Skip to content

DOC Improve doc about joblib warnings#1367

Merged
larsoner merged 5 commits intosphinx-gallery:masterfrom
lucyleeow:doc_joblib
Aug 3, 2024
Merged

DOC Improve doc about joblib warnings#1367
larsoner merged 5 commits intosphinx-gallery:masterfrom
lucyleeow:doc_joblib

Conversation

@lucyleeow
Copy link
Copy Markdown
Contributor

Follow on from #1362

Python warnings emitted by :mod:`joblib` during gallery generation (e.g., the
``UserWarning`` about a
`worker restarting <https://github.com/joblib/joblib/issues/883>`_),
will be captured by Sphinx-Gallery. These can be filtered out with
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Also not sure if these warnings are included in code execution output, so I've left at a vague 'capture'...?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Actually thinking again, it may be to do with when the warning is emitted, at the time of gallery generation/example execution which is why it causes examples to fail (when warnings are error)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I've amended the doc, hopefully it is right and makes sense

Copy link
Copy Markdown
Contributor

@larsoner larsoner left a comment

Choose a reason for hiding this comment

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

Nice, thanks @lucyleeow !

@larsoner larsoner enabled auto-merge (squash) August 3, 2024 13:50
@larsoner larsoner merged commit b1ff40a into sphinx-gallery:master Aug 3, 2024
@lucyleeow lucyleeow deleted the doc_joblib branch August 4, 2024 05:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants