healtex / texscrubber

Personal information de-identification tool

Apache License 2.0

2 stars 2 forks source link

[Feature] Workflow error handling and sharing data between steps #27

Closed hkkenneth closed 7 years ago

hkkenneth commented 7 years ago

8 .. It is one of the many ways to do it.. a bit opinionated here.

It is without proper error logging now, do we want to log to a separate file, or log to the output file itself? (e.g. if pat01-01.txt failed, when I open pat01-01.txt in the output folder, it may contain "this file has failed anonymization" literally as content)?

@RichJackson we don't seem to need ExecutionContext for this?

RichJackson commented 7 years ago

I'm going to merge this in the name of keeping things moving. A couple of points:

@hkkenneth - you're right, I don't see the use of an execution context at this stage

usage instructions:

"GATE home", "workspace/sample-input", "workspace/dev-text-output" and "workspace/dev-test-output-2" needs to be in the same dir as where the application is launched from, other wise you get NPE's