svn commit: r1197746 - /uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae/controller/PrimitiveAnalysisEngineController_impl.java

2011-11-04 Thread cwiklik
Author: cwiklik
Date: Fri Nov  4 20:23:36 2011
New Revision: 1197746

URL: http://svn.apache.org/viewvc?rev=1197746&view=rev
Log:
UIMA-2180 When generating per CAS stats skip the top level aggregate stats, and 
only include primitives

Modified:

uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae/controller/PrimitiveAnalysisEngineController_impl.java

Modified: 
uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae/controller/PrimitiveAnalysisEngineController_impl.java
URL: 
http://svn.apache.org/viewvc/uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae/controller/PrimitiveAnalysisEngineController_impl.java?rev=1197746&r1=1197745&r2=1197746&view=diff
==
--- 
uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae/controller/PrimitiveAnalysisEngineController_impl.java
 (original)
+++ 
uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae/controller/PrimitiveAnalysisEngineController_impl.java
 Fri Nov  4 20:23:36 2011
@@ -538,9 +538,10 @@ public class PrimitiveAnalysisEngineCont
   
   
   AnalysisEngineManagement rootAem = ae.getManagementInterface();
-  beforeAnalysisManagementObjects.add(deepCopy(rootAem));   
   if ( rootAem.getComponents().size() > 0 ) {
   getLeafManagementObjects(rootAem, beforeAnalysisManagementObjects, 
true);
+  } else {
+  beforeAnalysisManagementObjects.add(deepCopy(rootAem));   
   }
   
   CasIterator casIterator = ae.processAndOutputNewCASes(aCAS);
@@ -795,13 +796,14 @@ public class PrimitiveAnalysisEngineCont
   //  These stats are internally maintained in a Map. If the AE is an 
aggregate
   //  the Map will contain AnalysisEngineManagement instance for each AE.
   AnalysisEngineManagement aem = ae.getManagementInterface();
-  //  Add the top level AnalysisEngineManagement instance.
-  afterAnalysisManagementObjects.add(aem);
   if ( aem.getComponents().size() > 0) {
   //  Flatten the hierarchy by recursively (if this AE is an 
aggregate) extracting  
   //  primitive AE's AnalysisEngineManagement instance and placing it 
in 
   //  afterAnalysisManagementObjects List.
   getLeafManagementObjects(aem, afterAnalysisManagementObjects, false);
+  } else {
+  //  Add the top level AnalysisEngineManagement instance.
+  afterAnalysisManagementObjects.add(aem);
   }
 
   //  Create a List to hold per CAS analysisTime and total number of CASes 
processed




svn commit: r1197584 - /uima/uimaj/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/ICasEditorInputListener.java

2011-11-04 Thread pkluegl
Author: pkluegl
Date: Fri Nov  4 14:41:59 2011
New Revision: 1197584

URL: http://svn.apache.org/viewvc?rev=1197584&view=rev
Log:
UIMA-2273
extended javadoc

Modified:

uima/uimaj/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/ICasEditorInputListener.java

Modified: 
uima/uimaj/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/ICasEditorInputListener.java
URL: 
http://svn.apache.org/viewvc/uima/uimaj/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/ICasEditorInputListener.java?rev=1197584&r1=1197583&r2=1197584&view=diff
==
--- 
uima/uimaj/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/ICasEditorInputListener.java
 (original)
+++ 
uima/uimaj/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/ICasEditorInputListener.java
 Fri Nov  4 14:41:59 2011
@@ -23,6 +23,9 @@ public interface ICasEditorInputListener
 
   /**
* This method is called if an {@link ICasDocument} is exchanged.
+   * The arguments of this methods can be null under certain circumstances.
+   * For example, if a document is opened where the type system cannot be 
found 
+   * for, then the new document will be null.
* 
* @param oldDocument
*  - the replaced, old document {@link ICasDocument}.




svn commit: r1197547 - in /uima/uimaj/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor: ICasEditor.java editview/FeatureStructureContentProvider.java fsview/FeatureStructureBro

2011-11-04 Thread pkluegl
Author: pkluegl
Date: Fri Nov  4 13:18:24 2011
New Revision: 1197547

URL: http://svn.apache.org/viewvc?rev=1197547&view=rev
Log:
UIMA-2273
added listener methods to ICasEditor and removed casts to AnnotationEditor in 
two views

Modified:

uima/uimaj/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/ICasEditor.java

uima/uimaj/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/editview/FeatureStructureContentProvider.java

uima/uimaj/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/fsview/FeatureStructureBrowserViewPage.java

Modified: 
uima/uimaj/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/ICasEditor.java
URL: 
http://svn.apache.org/viewvc/uima/uimaj/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/ICasEditor.java?rev=1197547&r1=1197546&r2=1197547&view=diff
==
--- 
uima/uimaj/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/ICasEditor.java
 (original)
+++ 
uima/uimaj/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/ICasEditor.java
 Fri Nov  4 13:18:24 2011
@@ -36,4 +36,9 @@ public interface ICasEditor extends IEdi
   CasDocumentProvider getCasDocumentProvider();
   
   void reopenEditorWithNewTypeSystem();
+  
+  void addCasEditorInputListener(ICasEditorInputListener listener);
+  
+  void removeCasEditorInputListener(ICasEditorInputListener listener);
+  
 }

Modified: 
uima/uimaj/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/editview/FeatureStructureContentProvider.java
URL: 
http://svn.apache.org/viewvc/uima/uimaj/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/editview/FeatureStructureContentProvider.java?rev=1197547&r1=1197546&r2=1197547&view=diff
==
--- 
uima/uimaj/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/editview/FeatureStructureContentProvider.java
 (original)
+++ 
uima/uimaj/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/editview/FeatureStructureContentProvider.java
 Fri Nov  4 13:18:24 2011
@@ -29,7 +29,6 @@ import org.apache.uima.cas.FeatureStruct
 import org.apache.uima.cas.StringArrayFS;
 import org.apache.uima.cas.Type;
 import org.apache.uima.caseditor.editor.AbstractDocumentListener;
-import org.apache.uima.caseditor.editor.AnnotationEditor;
 import org.apache.uima.caseditor.editor.ArrayValue;
 import org.apache.uima.caseditor.editor.CasEditorError;
 import org.apache.uima.caseditor.editor.FeatureValue;
@@ -55,9 +54,7 @@ final class FeatureStructureContentProvi
 if (mDocument == null) {
   throw new IllegalArgumentException("document parameter must not be 
null!");
 }
-if(mEditor instanceof AnnotationEditor) {
-  ((AnnotationEditor)mEditor).addCasEditorInputListener(this);
-}
+ mEditor.addCasEditorInputListener(this);
   }
 
   private int arraySize(FeatureStructure value) {
@@ -123,9 +120,7 @@ final class FeatureStructureContentProvi
 
   public void dispose() {
 mDocument.removeChangeListener(this);
-if(mEditor instanceof AnnotationEditor) {
-  ((AnnotationEditor)mEditor).removeCasEditorInputListener(this);
-}
+mEditor.removeCasEditorInputListener(this);
   }
 
   public void inputChanged(final Viewer viewer, Object oldInput, Object 
newInput) {

Modified: 
uima/uimaj/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/fsview/FeatureStructureBrowserViewPage.java
URL: 
http://svn.apache.org/viewvc/uima/uimaj/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/fsview/FeatureStructureBrowserViewPage.java?rev=1197547&r1=1197546&r2=1197547&view=diff
==
--- 
uima/uimaj/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/fsview/FeatureStructureBrowserViewPage.java
 (original)
+++ 
uima/uimaj/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/fsview/FeatureStructureBrowserViewPage.java
 Fri Nov  4 13:18:24 2011
@@ -33,7 +33,6 @@ import org.apache.uima.cas.text.Annotati
 import org.apache.uima.caseditor.CasEditorPlugin;
 import org.apache.uima.caseditor.Images;
 import org.apache.uima.caseditor.editor.AbstractAnnotationDocumentListener;
-import org.apache.uima.caseditor.editor.AnnotationEditor;
 import org.apache.uima.caseditor.editor.FeatureValue;
 import org.apache.uima.caseditor.editor.ICasDocument;
 import org.apache.uima.caseditor.editor.ICasEditor;
@@ -81,9 +80,7 @@ public final class FeatureStructureBrows
 FeatureStructureTreeContentProvider(ICasEditor editor) {
   mEditor = editor;
   mDocument = editor.getDocument();
-  if(mEditor instanceof AnnotationEditor) {
-((AnnotationEditor)mEditor).addCasEditorInputListener(this);

svn commit: r1197545 - in /uima: sandbox/trunk/TextMarker/uimaj-ep-textmarker-addons/src/main/java/org/apache/uima/textmarker/explain/apply/ sandbox/trunk/TextMarker/uimaj-ep-textmarker-addons/src/mai

2011-11-04 Thread pkluegl
Author: pkluegl
Date: Fri Nov  4 13:13:43 2011
New Revision: 1197545

URL: http://svn.apache.org/viewvc?rev=1197545&view=rev
Log:
UIMA-2273
The notification of views when the cas document is exchanged will now be 
processed by a new listener and controlled by the AnnotationEditor in the 
doSetInput method. Adapted known view. I hope I haven't missed any. The update 
logic should be improved by using setInput (see comments on jira)...

Added:

uima/uimaj/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/ICasEditorInputListener.java
Modified:

uima/sandbox/trunk/TextMarker/uimaj-ep-textmarker-addons/src/main/java/org/apache/uima/textmarker/explain/apply/ApplyViewPage.java

uima/sandbox/trunk/TextMarker/uimaj-ep-textmarker-addons/src/main/java/org/apache/uima/textmarker/testing/ui/views/EvaluationViewPage.java

uima/sandbox/trunk/TextMarker/uimaj-ep-textmarker-caseditor/src/main/java/org/apache/uima/textmarker/caseditor/view/tree/AnnotationTreeContentProvider.java

uima/uimaj/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/AbstractDocument.java

uima/uimaj/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/AnnotationDocument.java

uima/uimaj/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/AnnotationEditor.java

uima/uimaj/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/DocumentUimaImpl.java

uima/uimaj/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/ICasDocument.java

uima/uimaj/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/ICasDocumentListener.java

uima/uimaj/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/editview/EditViewPage.java

uima/uimaj/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/editview/FeatureStructureContentProvider.java

uima/uimaj/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/fsview/FeatureStructureBrowserViewPage.java

uima/uimaj/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/outline/OutlineContentProviderBase.java

Modified: 
uima/sandbox/trunk/TextMarker/uimaj-ep-textmarker-addons/src/main/java/org/apache/uima/textmarker/explain/apply/ApplyViewPage.java
URL: 
http://svn.apache.org/viewvc/uima/sandbox/trunk/TextMarker/uimaj-ep-textmarker-addons/src/main/java/org/apache/uima/textmarker/explain/apply/ApplyViewPage.java?rev=1197545&r1=1197544&r2=1197545&view=diff
==
--- 
uima/sandbox/trunk/TextMarker/uimaj-ep-textmarker-addons/src/main/java/org/apache/uima/textmarker/explain/apply/ApplyViewPage.java
 (original)
+++ 
uima/sandbox/trunk/TextMarker/uimaj-ep-textmarker-addons/src/main/java/org/apache/uima/textmarker/explain/apply/ApplyViewPage.java
 Fri Nov  4 13:13:43 2011
@@ -19,15 +19,13 @@
 
 package org.apache.uima.textmarker.explain.apply;
 
-import java.util.Collection;
 import java.util.HashMap;
 import java.util.Map;
 
-import org.apache.uima.cas.text.AnnotationFS;
-import org.apache.uima.caseditor.editor.AbstractAnnotationDocumentListener;
 import org.apache.uima.caseditor.editor.AnnotationEditor;
 import org.apache.uima.caseditor.editor.ICasDocument;
 import org.apache.uima.caseditor.editor.ICasDocumentListener;
+import org.apache.uima.caseditor.editor.ICasEditorInputListener;
 import org.apache.uima.textmarker.addons.TextMarkerAddonsPlugin;
 import org.apache.uima.textmarker.explain.ExplainConstants;
 import org.apache.uima.textmarker.explain.tree.ExplainTree;
@@ -45,7 +43,7 @@ import org.eclipse.ui.IWorkbenchPart;
 import org.eclipse.ui.part.IPageSite;
 import org.eclipse.ui.part.Page;
 
-public class ApplyViewPage extends Page implements ISelectionListener {
+public class ApplyViewPage extends Page implements ISelectionListener, 
ICasEditorInputListener {
 
   protected TreeViewer viewer;
 
@@ -123,41 +121,7 @@ public class ApplyViewPage extends Page 
 viewer.setInput(tree.getRoot());
 getSite().setSelectionProvider(viewer);
 getSite().getPage().addSelectionListener(this);
-
-listener = new AbstractAnnotationDocumentListener() {
-
-  @Override
-  public void changed() {
-  }
-
-  @Override
-  public void viewChanged(String oldViewName, String newViewName) {
-  }
-
-  @Override
-  protected void addedAnnotation(Collection annotations) {
-  }
-
-  @Override
-  protected void removedAnnotation(Collection annotations) {
-  }
-
-  @Override
-  protected void updatedAnnotation(Collection annotations) {
-  }
-
-  @Override
-  public void casDocumentChanged(ICasDocument oldDocument, ICasDocument 
newDocument) {
-document.removeChangeListener(this);
-document = newDocument;
-document.addChangeListener(this);
-ExplainTree tree = new ExplainTree(document.getCAS());
-view