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

TST: remove IPython warning in debug session #13657

Merged
merged 1 commit into from Mar 9, 2021

Conversation

tupui
Copy link
Member

@tupui tupui commented Mar 8, 2021

What does this implement/fix?

When starting a debug session in PyCharm, the console cannot be used due to IPython warnings being caught.

Traceback (most recent call last):
  File "/Applications/PyCharm.app/Contents/plugins/python/helpers/pydev/_pydevd_bundle/pydevd_console_integration.py", line 84, in get_code_executor
    code_executor = getattr(__builtin__, 'interpreter')
AttributeError: module 'builtins' has no attribute 'interpreter'
During handling of the above exception, another exception occurred:
...
DeprecationWarning: The `use_readline` parameter is deprecated and ignored since IPython 6.0.

The fix consist in skipping the warning in the pytest configuration. Basically it's the first thing I do when I create a new branch to work on.

Additional information

This is also done here: mne-tools/mne-python#8524

@tupui tupui changed the title MAINT: remove IPython warning in debug session TST: remove IPython warning in debug session Mar 8, 2021
@tupui tupui added the maintenance Items related to regular maintenance tasks label Mar 8, 2021
Copy link
Member

@rgommers rgommers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks @tupui

@rgommers rgommers merged commit 9ebe127 into scipy:master Mar 9, 2021
@rgommers rgommers added this to the 1.7.0 milestone Mar 9, 2021
@tupui
Copy link
Member Author

tupui commented Mar 9, 2021

Great thanks!

@tupui tupui deleted the pytest_warning branch March 30, 2021 06:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Items related to regular maintenance tasks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants