Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

use logging #34

Closed
orbeckst opened this issue Nov 1, 2017 · 4 comments
Closed

use logging #34

orbeckst opened this issue Nov 1, 2017 · 4 comments

Comments

@orbeckst
Copy link
Member

orbeckst commented Nov 1, 2017

For debug and info output we do not want to use print but use the standard logging facility.

@orbeckst
Copy link
Member Author

orbeckst commented Nov 1, 2017

See #32 (comment) for more discussion.

@orbeckst
Copy link
Member Author

We now also use logging for MBAR (see PR #172 ).

@orbeckst
Copy link
Member Author

orbeckst commented Oct 29, 2021

There are currently no print() function calls in alchemlyb itself.

To close this issue, we only need to add to the docs

  • user: how to enable logging
  • API doc: use logger with name "alchemlyb.MODULE.CLASS" name space.

We can then keep adding logging where necessary.

@orbeckst
Copy link
Member Author

orbeckst commented Jun 6, 2023

With PR #303 we now have docs for logging.

@orbeckst orbeckst closed this as completed Jun 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants