8000 Full documentation build fails since set_output('pandas') was merged · Issue #24652 · scikit-learn/scikit-learn · GitHub
[go: up one dir, main page]
Skip to content

Full documentation build fails since set_output('pandas') was merged #24652

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.

Sign up for GitHub

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

Jump to bottom
Closed
lesteve opened this issue Oct 13, 2022 · 0 comments · Fixed by #24654
Closed

Full documentation build fails since set_output('pandas') was merged #24652

lesteve opened this issue Oct 13, 2022 · 0 comments · Fixed by #24654
Labels
Bug Build / CI

Comments

@lesteve
Copy link
Member
lesteve commented Oct 13, 2022 •
edited
Loading

https://github.com/scikit-learn/scikit-learn/actions/workflows/build-docs.yml?query=branch%3Amain

To reproduce locally:

EXAMPLES_PATTERN='plot_set_output|plot_scaling_importance' make html

The problem comes from the fact that examples are run in the same interpreter and may have side-effects on each other. Examples running after plot_set_output.py will have their transform_output set to 'pandas' and some fail because of this. Is this actually expected that some example fail if transform_output='pandas', I am not sure ...

The easiest fix is probably to reset the default output at the end of plot_set_output
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug Build / CI
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

MAINT Fix full doc build by avoiding plot_set_output.py side-effect lesteve/scikit-learn
1 participant
@lesteve
0