Skip to content

allow category scatter plots on existing axes - #1241

Open
gheshm-jpg wants to merge 2 commits into
rasbt:masterfrom
gheshm-jpg:fix-category-scatter-axes
Open

gheshm-jpg wants to merge 2 commits into
rasbt:masterfrom
gheshm-jpg:fix-category-scatter-axes

Conversation

@gheshm-jpg

@gheshm-jpg gheshm-jpg commented Oct 10, 2026 •

Copy link
Copy Markdown
Contributor

description

added ax to category_scatter so it can draw into existing subplots, with a side-by-side example. calls without ax still create a new figure. both linux ci jobs explicitly run the new tests because normal collection skips plotting tests. fixes #636.

checks

  • regression tests, notebook and changelog updated
  • four new axes cases fail before the fix; all five focused tests pass afterward
  • all notebook cells executed; converted docs and checked the plot
  • plotting flake8, changed-file black/isort and git diff --check passed

25 plotting tests pass with test_pca_corr_graph.py excluded; that file blocks collection with an existing Version < str error. full-site build was not rerun because of the known Markdown-extension incompatibility.

@review-notebook-app

Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

This branch has not been deployed

No deployments
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.

How to 2 plots side by side?

1 participant