Duplicate logging information about geometric HOT 4 CLOSED

Hi Daniel,

Those changes were made around the time I reconfigured the geomeTRIC loggers to require an explicit newline character for every logging message, as I had some situations where I wanted finer control over whether newline characters were being printed or not.

I think replacing handlers=stream_handler with handlers= in line 12 of logJson.ini might also resolve the issue, though it disables printing to stdout when I run the unit tests.

Is there any way I could see the stdout from running the unit tests without duplicating the logging on your side? If you could just show me how to print out the string stream in the unit test codes, that should resolve it.

from geometric.

For running through the QCFractal pipeline, we would like all stdout to be contained within with stdout field. If you run the JSON test you can have a print(ret["stdout"]) or similar line. The way we had this previously (I believe) was that the run_json would temporarily remove the old handler which likely dumped to stdout and replace it with it with a handler that was based of StringIO and replace the old handle when complete.

You could consider an option on the run_json command to turn back on stdout stream printing while running tests.

from geometric.

Release 0.9.7.2 is now out, could you please check to see if it resolved our two logging issues (as well as the other connectivity-related issue in your PR)?

from geometric.

Perfect on our end! Thanks for the quick turnaround.

from geometric.