You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@uima.apache.org by pk...@apache.org on 2012/01/10 14:18:50 UTC

svn commit: r1229526 [1/2] - in /uima/sandbox/trunk/TextMarker/uimaj-ep-textmarker-engine/src/main/java/org/apache/uima/textmarker: TextMarkerEnvironment.java action/SetFeatureAction.java parser/TextMarkerParser.g parser/TextMarkerParser.java

Author: pkluegl
Date: Tue Jan 10 13:18:50 2012
New Revision: 1229526

URL: http://svn.apache.org/viewvc?rev=1229526&view=rev
Log:
UIMA-2325
added functionality to remember and set initial  values of primitive variables.

Modified:
    uima/sandbox/trunk/TextMarker/uimaj-ep-textmarker-engine/src/main/java/org/apache/uima/textmarker/TextMarkerEnvironment.java
    uima/sandbox/trunk/TextMarker/uimaj-ep-textmarker-engine/src/main/java/org/apache/uima/textmarker/action/SetFeatureAction.java
    uima/sandbox/trunk/TextMarker/uimaj-ep-textmarker-engine/src/main/java/org/apache/uima/textmarker/parser/TextMarkerParser.g
    uima/sandbox/trunk/TextMarker/uimaj-ep-textmarker-engine/src/main/java/org/apache/uima/textmarker/parser/TextMarkerParser.java

Modified: uima/sandbox/trunk/TextMarker/uimaj-ep-textmarker-engine/src/main/java/org/apache/uima/textmarker/TextMarkerEnvironment.java
URL: http://svn.apache.org/viewvc/uima/sandbox/trunk/TextMarker/uimaj-ep-textmarker-engine/src/main/java/org/apache/uima/textmarker/TextMarkerEnvironment.java?rev=1229526&r1=1229525&r2=1229526&view=diff
==============================================================================
--- uima/sandbox/trunk/TextMarker/uimaj-ep-textmarker-engine/src/main/java/org/apache/uima/textmarker/TextMarkerEnvironment.java (original)
+++ uima/sandbox/trunk/TextMarker/uimaj-ep-textmarker-engine/src/main/java/org/apache/uima/textmarker/TextMarkerEnvironment.java Tue Jan 10 13:18:50 2012
@@ -36,13 +36,16 @@ import org.apache.uima.jcas.cas.TOP;
 import org.apache.uima.jcas.tcas.DocumentAnnotation;
 import org.apache.uima.textmarker.action.AbstractTextMarkerAction;
 import org.apache.uima.textmarker.condition.AbstractTextMarkerCondition;
+import org.apache.uima.textmarker.expression.bool.BooleanExpression;
 import org.apache.uima.textmarker.expression.list.ListExpression;
 import org.apache.uima.textmarker.expression.list.SimpleBooleanListExpression;
 import org.apache.uima.textmarker.expression.list.SimpleNumberListExpression;
 import org.apache.uima.textmarker.expression.list.SimpleStringListExpression;
 import org.apache.uima.textmarker.expression.list.SimpleTypeListExpression;
+import org.apache.uima.textmarker.expression.number.NumberExpression;
 import org.apache.uima.textmarker.expression.resource.LiteralWordListExpression;
 import org.apache.uima.textmarker.expression.resource.LiteralWordTableExpression;
+import org.apache.uima.textmarker.expression.string.StringExpression;
 import org.apache.uima.textmarker.resource.CSVTable;
 import org.apache.uima.textmarker.resource.MultiTreeWordList;
 import org.apache.uima.textmarker.resource.TextMarkerTable;
@@ -77,6 +80,8 @@ public class TextMarkerEnvironment {
 
   private CAS cas;
 
+  private Map<String, Object> initializedVariables;
+
   public TextMarkerEnvironment(TextMarkerBlock owner) {
     super();
     this.owner = owner;
@@ -91,7 +96,7 @@ public class TextMarkerEnvironment {
     availableTypes.put("INT", Integer.class);
     availableTypes.put("STRING", String.class);
     availableTypes.put("DOUBLE", Double.class);
-    availableTypes.put("FLOAT", Double.class);
+    availableTypes.put("FLOAT", Float.class);
     availableTypes.put("BOOLEAN", Boolean.class);
     availableTypes.put("TYPE", Type.class);
     availableTypes.put("CONDITION", AbstractTextMarkerCondition.class);
@@ -112,6 +117,7 @@ public class TextMarkerEnvironment {
     availableListTypes.put("STRINGLIST", String.class);
     availableListTypes.put("TYPELIST", Type.class);
     resourcePaths = getResourcePaths();
+    initializedVariables = new HashMap<String, Object>();
   }
 
   public void initializeTypes(CAS cas) {
@@ -239,10 +245,14 @@ public class TextMarkerEnvironment {
     if (generic != null) {
       variableGenericTypes.put(name, generic);
     }
-    variableValues.put(name, getInitialValue(type));
+    variableValues.put(name, getInitialValue(name, type));
   }
 
-  private Object getInitialValue(Class<?> type) {
+  private Object getInitialValue(String name, Class<?> type) {
+    Object init = initializedVariables.get(name);
+    if (init != null) {
+      return init;
+    }
     if (Integer.class.equals(type)) {
       return 0;
     } else if (Double.class.equals(type)) {
@@ -262,7 +272,6 @@ public class TextMarkerEnvironment {
     } else if (List.class.equals(type)) {
       return new ArrayList<Object>();
     }
-
     return null;
   }
 
@@ -322,7 +331,7 @@ public class TextMarkerEnvironment {
     if (containsKey && result == null) {
       // TODO find the problem with the null values!
       // this might now work for word lists in another env.
-      return type.cast(getInitialValue(type));
+      return type.cast(getInitialValue(name, type));
     }
     if (result == annotationTypeDummy) {
       return type.cast(cas.getAnnotationType());
@@ -339,6 +348,42 @@ public class TextMarkerEnvironment {
     return getVariableValue(name, Object.class);
   }
 
+  public Object getLiteralValue(String var, Object value) {
+    if (ownsVariable(var)) {
+      Class<?> clazz = variableTypes.get(var);
+      if (value instanceof NumberExpression) {
+        NumberExpression ne = (NumberExpression) value;
+        if (clazz.equals(Integer.class)) {
+          return ne.getIntegerValue(owner);
+        } else if (clazz.equals(Double.class)) {
+          return ne.getDoubleValue(owner);
+        } else if (clazz.equals(Float.class)) {
+          return ne.getFloatValue(owner);
+        } else if (clazz.equals(String.class)) {
+          return ne.getStringValue(owner);
+        }
+      } else if (clazz.equals(String.class) && value instanceof StringExpression) {
+        StringExpression se = (StringExpression) value;
+        return se.getStringValue(owner);
+      } else if (clazz.equals(Boolean.class) && value instanceof BooleanExpression) {
+        BooleanExpression be = (BooleanExpression) value;
+        return be.getBooleanValue(owner);
+      }
+      return null;
+    } else {
+      return owner.getParent().getEnvironment().getLiteralValue(var, value);
+    }
+  }
+
+  public void setInitialVariableValue(String var, Object value) {
+    if (ownsVariable(var)) {
+      initializedVariables.put(var, value);
+      setVariableValue(var, value);
+    } else if (owner.getParent() != null) {
+      owner.getParent().getEnvironment().setInitialVariableValue(var, value);
+    }
+  }
+
   public void setVariableValue(String var, Object value) {
     if (ownsVariable(var)) {
       Class<?> clazz = variableTypes.get(var);
@@ -363,7 +408,7 @@ public class TextMarkerEnvironment {
         variableValues.put(var, list);
       } else {
         if (value == null) {
-          value = getInitialValue(clazz);
+          value = getInitialValue(var, clazz);
         }
         variableValues.put(var, value);
       }
@@ -394,7 +439,7 @@ public class TextMarkerEnvironment {
     Set<Entry<String, Object>> entrySet = variableValues.entrySet();
     for (Entry<String, Object> entry : entrySet) {
       String key = entry.getKey();
-      Object initialValue = getInitialValue(variableTypes.get(key));
+      Object initialValue = getInitialValue(key, variableTypes.get(key));
       if (initialValue != null) {
         // not for word lists
         entry.setValue(initialValue);

Modified: uima/sandbox/trunk/TextMarker/uimaj-ep-textmarker-engine/src/main/java/org/apache/uima/textmarker/action/SetFeatureAction.java
URL: http://svn.apache.org/viewvc/uima/sandbox/trunk/TextMarker/uimaj-ep-textmarker-engine/src/main/java/org/apache/uima/textmarker/action/SetFeatureAction.java?rev=1229526&r1=1229525&r2=1229526&view=diff
==============================================================================
--- uima/sandbox/trunk/TextMarker/uimaj-ep-textmarker-engine/src/main/java/org/apache/uima/textmarker/action/SetFeatureAction.java (original)
+++ uima/sandbox/trunk/TextMarker/uimaj-ep-textmarker-engine/src/main/java/org/apache/uima/textmarker/action/SetFeatureAction.java Tue Jan 10 13:18:50 2012
@@ -127,7 +127,7 @@ public class SetFeatureAction extends Ab
             double v = numberExpr.getDoubleValue(element.getParent());
             annotationFS.setDoubleValue(featureByBaseName, v);
           } else if (range.equals(UIMAConstants.TYPE_FLOAT)) {
-            float v = (float) numberExpr.getDoubleValue(element.getParent());
+            float v = (float) numberExpr.getFloatValue(element.getParent());
             annotationFS.setFloatValue(featureByBaseName, v);
           } else if (range.equals(UIMAConstants.TYPE_BYTE)) {
             byte v = (byte) numberExpr.getIntegerValue(element.getParent());

Modified: uima/sandbox/trunk/TextMarker/uimaj-ep-textmarker-engine/src/main/java/org/apache/uima/textmarker/parser/TextMarkerParser.g
URL: http://svn.apache.org/viewvc/uima/sandbox/trunk/TextMarker/uimaj-ep-textmarker-engine/src/main/java/org/apache/uima/textmarker/parser/TextMarkerParser.g?rev=1229526&r1=1229525&r2=1229526&view=diff
==============================================================================
--- uima/sandbox/trunk/TextMarker/uimaj-ep-textmarker-engine/src/main/java/org/apache/uima/textmarker/parser/TextMarkerParser.g (original)
+++ uima/sandbox/trunk/TextMarker/uimaj-ep-textmarker-engine/src/main/java/org/apache/uima/textmarker/parser/TextMarkerParser.g Tue Jan 10 13:18:50 2012
@@ -122,9 +122,17 @@ public void emitErrorMessage(String msg)
 		return parent.getEnvironment().isVariable(name);
 	}
 	
-	public void setValue(TextMarkerBlock parent, String name, Object value) {
-		if(value != null) {
+	public void setValue(TextMarkerBlock parent, List<String> names, Object obj) {
+		for(String name : names) {
+			setValue(parent,name,obj);
+		}
+	}
+	
+	public void setValue(TextMarkerBlock parent, String name, Object obj) {
+		if(obj != null) {
+			Object value = parent.getEnvironment().getLiteralValue(name, obj);
 			parent.getEnvironment().setVariableValue(name, value);
+			parent.getEnvironment().setInitialVariableValue(name, value);
 		}
 	}
 	
@@ -247,36 +255,39 @@ statement returns [TextMarkerStatement s
 	;
 
 variableDeclaration returns [TextMarkerStatement stmt = null]
+@init {
+List<String> vars = new ArrayList<String>();
+}
 	:
 	type = IntString
-	{!ownsVariable($blockDeclaration::env, input.LT(1).getText())}? id = Identifier {addVariable($blockDeclaration::env, id.getText(), type.getText());}
-		(COMMA {!ownsVariable($blockDeclaration::env, input.LT(1).getText())}? id = Identifier {addVariable($blockDeclaration::env, id.getText(), type.getText());}
-		 )*(ASSIGN_EQUAL value1 = numberExpression)? {setValue($blockDeclaration::env, id.getText(), value1);}  SEMI
+	{!ownsVariable($blockDeclaration::env, input.LT(1).getText())}? id = Identifier {vars.add(id.getText());addVariable($blockDeclaration::env, id.getText(), type.getText());}
+		(COMMA {!ownsVariable($blockDeclaration::env, input.LT(1).getText())}? id = Identifier {vars.add(id.getText());addVariable($blockDeclaration::env, id.getText(), type.getText());}
+		 )*(ASSIGN_EQUAL value1 = numberExpression)? {setValue($blockDeclaration::env, vars, value1);}  SEMI
 	|
 	type = DoubleString
-	{!ownsVariable($blockDeclaration::env, input.LT(1).getText())}? id = Identifier {addVariable($blockDeclaration::env, id.getText(), type.getText());}
-		(COMMA {!ownsVariable($blockDeclaration::env, input.LT(1).getText())}? id = Identifier {addVariable($blockDeclaration::env, id.getText(), type.getText());}
-		 )* (ASSIGN_EQUAL value2 = numberExpression)? {setValue($blockDeclaration::env, id.getText(), value2);} SEMI
+	{!ownsVariable($blockDeclaration::env, input.LT(1).getText())}? id = Identifier {vars.add(id.getText());addVariable($blockDeclaration::env, id.getText(), type.getText());}
+		(COMMA {!ownsVariable($blockDeclaration::env, input.LT(1).getText())}? id = Identifier {vars.add(id.getText());addVariable($blockDeclaration::env, id.getText(), type.getText());}
+		 )* (ASSIGN_EQUAL value2 = numberExpression)? {setValue($blockDeclaration::env, vars, value2);} SEMI
 	|
 	type = FloatString
-	{!ownsVariable($blockDeclaration::env, input.LT(1).getText())}? id = Identifier {addVariable($blockDeclaration::env, id.getText(), type.getText());}
-		(COMMA {!ownsVariable($blockDeclaration::env, input.LT(1).getText())}? id = Identifier {addVariable($blockDeclaration::env, id.getText(), type.getText());}
-		 )* (ASSIGN_EQUAL value2 = numberExpression)? {setValue($blockDeclaration::env, id.getText(), value2);} SEMI
+	{!ownsVariable($blockDeclaration::env, input.LT(1).getText())}? id = Identifier {vars.add(id.getText());addVariable($blockDeclaration::env, id.getText(), type.getText());}
+		(COMMA {!ownsVariable($blockDeclaration::env, input.LT(1).getText())}? id = Identifier {vars.add(id.getText());addVariable($blockDeclaration::env, id.getText(), type.getText());}
+		 )* (ASSIGN_EQUAL value2 = numberExpression)? {setValue($blockDeclaration::env, vars, value2);} SEMI
 	|
 	type = StringString
-	{!ownsVariable($blockDeclaration::env, input.LT(1).getText())}? id = Identifier {addVariable($blockDeclaration::env, id.getText(), type.getText());}
-		(COMMA {!ownsVariable($blockDeclaration::env, input.LT(1).getText())}? id = Identifier {addVariable($blockDeclaration::env, id.getText(), type.getText());}
-		 )* (ASSIGN_EQUAL value3 = stringExpression)? {setValue($blockDeclaration::env, id.getText(), value3);} SEMI
+	{!ownsVariable($blockDeclaration::env, input.LT(1).getText())}? id = Identifier {vars.add(id.getText());addVariable($blockDeclaration::env, id.getText(), type.getText());}
+		(COMMA {!ownsVariable($blockDeclaration::env, input.LT(1).getText())}? id = Identifier {vars.add(id.getText());addVariable($blockDeclaration::env, id.getText(), type.getText());}
+		 )* (ASSIGN_EQUAL value3 = stringExpression)? {setValue($blockDeclaration::env, vars, value3);} SEMI
 	|
 	type = BooleanString
-	{!ownsVariable($blockDeclaration::env, input.LT(1).getText())}? id = Identifier {addVariable($blockDeclaration::env, id.getText(), type.getText());}
-		(COMMA {!ownsVariable($blockDeclaration::env, input.LT(1).getText())}? id = Identifier {addVariable($blockDeclaration::env, id.getText(), type.getText());}
-		 )* (ASSIGN_EQUAL value4 = booleanExpression)? {setValue($blockDeclaration::env, id.getText(), value4);} SEMI
+	{!ownsVariable($blockDeclaration::env, input.LT(1).getText())}? id = Identifier {vars.add(id.getText());addVariable($blockDeclaration::env, id.getText(), type.getText());}
+		(COMMA {!ownsVariable($blockDeclaration::env, input.LT(1).getText())}? id = Identifier {vars.add(id.getText());addVariable($blockDeclaration::env, id.getText(), type.getText());}
+		 )* (ASSIGN_EQUAL value4 = booleanExpression)? {setValue($blockDeclaration::env, vars, value4);} SEMI
 	|
 	type = TypeString
-	{!ownsVariable($blockDeclaration::env, input.LT(1).getText())}? id = Identifier {addVariable($blockDeclaration::env, id.getText(), type.getText());}
-		(COMMA {!ownsVariable($blockDeclaration::env, input.LT(1).getText())}? id = Identifier {addVariable($blockDeclaration::env, id.getText(), type.getText());}
-		 )* (ASSIGN_EQUAL value5 = annotationType)? {setValue($blockDeclaration::env, id.getText(), value5);} SEMI
+	{!ownsVariable($blockDeclaration::env, input.LT(1).getText())}? id = Identifier {vars.add(id.getText());addVariable($blockDeclaration::env, id.getText(), type.getText());}
+		(COMMA {!ownsVariable($blockDeclaration::env, input.LT(1).getText())}? id = Identifier {vars.add(id.getText());addVariable($blockDeclaration::env, id.getText(), type.getText());}
+		 )* (ASSIGN_EQUAL value5 = annotationType)? {setValue($blockDeclaration::env, vars, value5);} SEMI
 	| 
 	type = WORDLIST 
 	{!isVariableOfType($blockDeclaration::env, input.LT(1).getText(), type.getText())}?