search

google-code-export / uimafit

Automatically exported from code.google.com/p/uimafit
2 stars 1 forks source link

support for different flow controllers in aggregate AEs #18

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
I received an inquiry and subsequent patch (attached) related to flow
controllers from Fabio Mancinelli.  I am going to take a look at it now. 
Here is the message he sent to me:

Fabio:
I am attaching a patch to UIMAFit for handling FlowControllers.

This patch doesn't allow the configuration of the flow controller
parameters but it should be just a matter of adding a bunch of
parameters to the createXXX methods (though making them a bit
heavyweight at the parameter list level).

The unit test is a bit ugly. I didn't find a way to test if the
flowcontroller was called without using a static variable.
I didn't have the time to write a nicer test... Maybe writing a
flowcontroller that calls only the second annotator and testing the
result would have been a more clean solution.
Maybe I'll spend some time on this asap.

Original issue reported on code.google.com by pvogren@gmail.com on 11 May 2010 at 4:40

Attachments:

GoogleCodeExporter commented 9 years ago

Original comment by pvogren@gmail.com on 11 May 2010 at 4:58