Skip to content

Conversation

@nat-salt
Copy link
Contributor

@nat-salt nat-salt commented Apr 6, 2022

Reference Issues/PRs

Fixes plot_pca_3d.py for #22406

What does this implement/fix? Explain your changes.

This updates plot_pca_3d.py to notebook style used in other examples

Any other comments?

@lesteve lesteve added the Quick Review For PRs that are quick to review label Apr 7, 2022
@lesteve lesteve mentioned this pull request Apr 7, 2022
47 tasks
@lesteve
Copy link
Member

lesteve commented Apr 8, 2022

The doc-min-dependencies build is failing. It looks like with the min matplotlib version you need to import import mpl_toolkits.mplot3d #22594 (comment)

@lesteve
Copy link
Member

lesteve commented Apr 8, 2022

I pushed some tweaks:

  • use underline of the right length
  • move imports in the cell they are first used
  • fix for matplotlib 3.1 mentioned in the previous comment

@lesteve lesteve changed the title updated plot_pca_3d.py DOC: use notebook-style for plot_pca_3d.py Apr 8, 2022
@lesteve
Copy link
Member

lesteve commented Apr 8, 2022

Merging, thanks a lot!

@lesteve lesteve merged commit fbc56ed into scikit-learn:main Apr 8, 2022
jjerphan pushed a commit to jjerphan/scikit-learn that referenced this pull request Apr 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Documentation Quick Review For PRs that are quick to review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants