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

Fix size norm extrapolation #2580

Merged
merged 5 commits into from
May 10, 2021
Merged

Fix size norm extrapolation #2580

merged 5 commits into from
May 10, 2021

Conversation

mwaskom
Copy link
Owner

@mwaskom mwaskom commented May 10, 2021

Builds on #2579 with

  • a test for the original bug
  • the addition of a size_range attribute on the SizeMapping object, rather than accessing the _default_size_range attribute, which is regrettable and will ideally get refactored away somehow.

@mwaskom mwaskom added this to the v0.12.0 milestone May 10, 2021
@codecov
Copy link

codecov bot commented May 10, 2021

Codecov Report

Merging #2580 (80e8efd) into master (4f79753) will increase coverage by 0.17%.
The diff coverage is 97.14%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2580      +/-   ##
==========================================
+ Coverage   97.27%   97.44%   +0.17%     
==========================================
  Files          17       17              
  Lines        6340     6351      +11     
==========================================
+ Hits         6167     6189      +22     
+ Misses        173      162      -11     
Impacted Files Coverage Δ
seaborn/rcmod.py 100.00% <ø> (ø)
seaborn/_core.py 98.47% <87.50%> (-0.15%) ⬇️
seaborn/categorical.py 98.53% <100.00%> (+0.14%) ⬆️
seaborn/matrix.py 95.87% <100.00%> (+<0.01%) ⬆️
seaborn/regression.py 97.88% <100.00%> (+0.71%) ⬆️
seaborn/distributions.py 96.37% <0.00%> (+0.32%) ⬆️
seaborn/_docstrings.py 100.00% <0.00%> (+12.50%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 536fa2d...80e8efd. Read the comment docs.

@mwaskom mwaskom merged commit 405f666 into master May 10, 2021
@mwaskom mwaskom deleted the fix_size_norm_extrapolation branch May 10, 2021 20:57
@mwaskom
Copy link
Owner Author

mwaskom commented Jul 17, 2021

This needs a release note! NVM it was caught in the next PR.

@mwaskom mwaskom modified the milestones: v0.12.0, v0.11.2 Aug 6, 2021
mwaskom added a commit that referenced this pull request Aug 6, 2021
* fix size range

* add default value to sizes

* Add test for GH2539

* Restrict usage of default size range attribute

* Work around archaic matplotlib auto-legending of named Series objects

Co-authored-by: Risako <hamarisa0623@chiba-u.jp>
(cherry picked from commit 405f666)
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.

1 participant