Frank Steggink

Several unit tests were implemented in pull requests #44 and #46. Coverage is now 36%.

The actual coverage turned out to be lower, since I didn't exclude the tests directory. Solved now in commit 194e780. Part of pull requests #48, including a few more unit...

I agree that in this specific test case an XmlElementStreamer file input is actually a more appropriate source. I also agree that the FileInput itself could become more generic (by...

As for the original use of the XmlElementReader followed by the XmlAssembler, this is the current TOP10NL workflow in NLExtract. The BGT and BRK are based on that as well....

The last issue (LCO) has been solved via #95. AFAIK, the rest still stands. When I'm in the occasion, I can submit a few PRs. I agree that we should...

More info: the replaced tildes already occur in the options_dict and args_dict which are passed to the ETL constructor. They're coming from the parse_args function in stetl.main. I suspect this...

The best solution depends on the workflow. I would keep Stetl as 'atomic' as possible. Just use it for a single task. IMO this means that it should be executed...

Yes, I've read about the various configuration options, and eventually figured out that by specifying a custom file through GHC_SETTINGS I can also override things like probe defaults. However I...

Another thing that surprised me (and I didn't try to find the cause of it yet) is that the file docker/compose/ghc.env doesn't try to specify all different config variables. Yet,...

Yes, that's right. I didn't think about any env vars set in the Dockerfile (or by some other means in the Docker container).