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

MAINT: Clear the clutter of warnings #838

Merged
merged 2 commits into from Apr 5, 2020

Conversation

oesteban
Copy link
Member

@oesteban oesteban commented Apr 5, 2020

The warnings are handled in two ways:

  • If this is a release, ignore all warnings (MRIQC does not use Python's warnings module, but logging instead).
  • If development, warnings.showwarning and warnings.warn are monkeypatched to redirect to the 'py.warnings' logger. The logger writes to /dev/null, but that can we change for debugging purposes.

Related: nipreps/fmriprep#1981

The warnings are handled in two ways:

  * If this is a release, ignore all warnings (MRIQC does not use
    Python's warnings module, but logging instead).
  * If development, warnings.showwarning and warnings.warn are
    monkeypatched to redirect to the ``'py.warnings'`` logger.
    The logger writes to /dev/null, but that can we change for
    debugging purposes.

Related: nipreps/fmriprep#1981
@oesteban oesteban requested a review from mgxd April 5, 2020 06:52
@oesteban oesteban merged commit be1da38 into nipreps:master Apr 5, 2020
@oesteban oesteban deleted the maint/remove-warnings-2 branch April 5, 2020 22:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant