-
Notifications
You must be signed in to change notification settings - Fork 12
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
Logging utilities #51
Conversation
Co-authored-by: Lander Burgelman <[email protected]>
…nberg test (failing), replace tsvd with QR in gauge fixing
Codecov ReportAttention: Patch coverage is
|
Looks good to me, but we should probably increase the verbosity in the examples and tests to at least |
I agree, that probably the default verbosity should be changed and that a quick explanation of levels 0 to 3 would be useful to have in the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Except for the verbosity in PEPSOptimize
I find this really nice and useful. Thanks!
This PR re-uses MPSKit functionality to handle the logging. I think it is a little cleaner, but I'm definitely open to suggestions.