-
Notifications
You must be signed in to change notification settings - Fork 22
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
ENH: make minimal traceback reporting optional #91
Conversation
Codecov Report
@@ Coverage Diff @@
## master #91 +/- ##
=======================================
Coverage 66.66% 66.66%
=======================================
Files 1 1
Lines 3 3
=======================================
Hits 2 2
Misses 1 1 Continue to review full report at Codecov.
|
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.
Looks good to me. Have a couple of minor suggestions.
@@ -92,7 +93,9 @@ def configure_base( | |||
True by default. Console output and exception stack traces will be | |||
written to IPython log file when IPython logging is enabled. | |||
publish_documents_to_kafka: boolean, optional | |||
False by default. If True publish bluesky documents to a Kafka message broker. | |||
False by default. If True publish bluesky documents to a Kafka message broker | |||
tb_minimize : boolean, optional |
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.
tb_minimize : boolean, optional | |
tb_minimize: boolean, optional |
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.
numpydoc says there should be a space before 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.
Hm, it's conflicting with what we discovered recently with @danielballan. Dan, do you remember where we saw issues with the latest Sphinx?
nslsii/__init__.py
Outdated
# always configure %xmode minimal | ||
# so short tracebacks are printed to the console | ||
get_ipython().magic("xmode minimal") | ||
# always |
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.
Was it left after moving comments around?
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.
oops.
It'd be nice if there were an exception stashing mechanism that worked like:
Of course |
@klauer That request should probably go to IPython |
172ba42
to
4c8431a
Compare
I think it exists: https://ipython.readthedocs.io/en/stable/interactive/magics.html#magic-tb |
Ah, not exactly. I somewhat misinterpreted @klauer’s comment on my initial reading. Nonetheless that might be interesting. |
Just throwing it out there 🤷♂️ |
4c8431a
to
7e1d1a8
Compare
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.
Needed for the queueserver work.
Because sometimes you do want to see the full tracebacks.