search

choderalab / perses

Experiments with expanded ensembles to explore chemical space
http://perses.readthedocs.io
MIT License
178 stars 50 forks source link

Make sure logging messages stream to stdout #1175

Closed ijpulidos closed 1 year ago

ijpulidos commented 1 year ago

By default the StreamHandler in logging outputs to sys.stderr, we should make sure that this gets streamed to sys.stdout instead, such that users get access to the logged information more easily.

Related to https://github.com/choderalab/openmmtools/pull/673

mikemhenry commented 1 year ago

I don't think we should change where we stream the logging messages, users can tell their submission system to combine the output into a single file if they want, or they can use the standard posix output direction ie perses-cli --yaml my.yaml 2>&1

ijpulidos commented 1 year ago

@mikemhenry I agree, I think this should be handled by the users at the higher level. Closing. Feel free to reopen if there's something else to be discussed here.