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

Need Context-dependent messaging facility #135

Open
eirrgang opened this issue Jul 11, 2018 · 1 comment
Open

Need Context-dependent messaging facility #135

eirrgang opened this issue Jul 11, 2018 · 1 comment

Comments

@eirrgang
Copy link
Collaborator

We desperately need a uniform facility to allow status message handling to be configured at a high level and to register handlers down into Sessions and API operations.

There is the potential for confusion about what "logging" is. Some common tools use "logging" to refer to status messages issued by code to be caught by an appropriate handler at different "levels", such as CRITICAL, WARN, STATUS, INFO, DEBUG. This should be distinguished from a facility for producing a record or artifact as a deliverable. I think it is helpful to adopt a convention of referring to configurable message handling as "message" and reserve "log" to refer to formulaic, deterministic output that might have stricter policies attached.

At the Python level, we would like the facility to be compatible with the logging module of Python. For starters, we can make a simple C++ adapter that we can pass through the Session to underlying code.

Some policy discussion is appropriate regarding messages across simulation ranks or MPI ranks.

@eirrgang
Copy link
Collaborator Author

We also need to think about the user-facing part of this. The way we currently configure the Python logging facility is pretty awkward.

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

1 participant