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

Add more robust logging #42

Open
alberdingk-thijm opened this issue Nov 26, 2019 · 2 comments
Open

Add more robust logging #42

alberdingk-thijm opened this issue Nov 26, 2019 · 2 comments
Labels
enhancement New feature or request P-low Low priority

Comments

@alberdingk-thijm
Copy link
Collaborator

A possible extension to the tool that would be handy from a development and debugging standpoint would be more robust logging. We could add a new dependency (e.g. logs) and use it to allow users to write logging information to stderr or to a file.

@alberdingk-thijm alberdingk-thijm added the enhancement New feature or request label Nov 26, 2019
@DKLoehr
Copy link
Collaborator

DKLoehr commented Nov 26, 2019

Can you be a little more specific? What sorts of logging do you think would be useful? If you just want to add the logs dependency, go ahead.

@alberdingk-thijm
Copy link
Collaborator Author

There are a couple of different forms logs could take. One option would be to print debug messages describing where we are in the control flow, e.g.:

[DEBUG] successfully parsed input
[DEBUG] running SMT with classical encoding
...

Another possibility would be to have logging describe what transformations are being run, what happens when certain functions are run, and so on. I don't consider this a high priority issue: more of something that might be nice to add for the off chance that someone wants to see what nv is doing without browsing the source code, or if someone is developing with nv and wants to inspect the control flow at a glance without needing to add a bunch of print statements (of course, if they want to do something more technical, they will need to dig into the code anyway).

@alberdingk-thijm alberdingk-thijm added the P-low Low priority label Sep 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request P-low Low priority
Projects
None yet
Development

No branches or pull requests

2 participants