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

Add Grid.figure to (eventually) replace Grid.fig #2639

Merged
merged 3 commits into from
Aug 14, 2021
Merged

Conversation

mwaskom
Copy link
Owner

@mwaskom mwaskom commented Aug 13, 2021

The advantage of this change is that you will be able to do

obj = sns.<func>()
obj.figure.<method>

regardless of whether <func> is figure-level or axes-level.

The Grid.fig methods are being soft-deprecated: discouraged in the
attribute documentation but not (currently) issuing a warning when used.

The advantage of this change is that you will be able to do

    obj = sns.<func>()
    obj.figure.<method>

regardless of whether  is figure-level or axes-level.

The `Grid.fig` methods are being soft-deprecated: discouraged in the
attribute documentation but not (currently) issuing a warning when used.
@codecov
Copy link

codecov bot commented Aug 13, 2021

Codecov Report

Merging #2639 (d6f53a9) into master (bfb7a04) will increase coverage by 0.07%.
The diff coverage is 100.00%.

❗ Current head d6f53a9 differs from pull request most recent head d7a7889. Consider uploading reports for the commit d7a7889 to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2639      +/-   ##
==========================================
+ Coverage   97.43%   97.51%   +0.07%     
==========================================
  Files          17       17              
  Lines        6405     6402       -3     
==========================================
+ Hits         6241     6243       +2     
+ Misses        164      159       -5     
Impacted Files Coverage Δ
seaborn/axisgrid.py 96.07% <100.00%> (+0.48%) ⬆️
seaborn/matrix.py 95.84% <100.00%> (-0.03%) ⬇️

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 bfb7a04...d7a7889. Read the comment docs.

@mwaskom mwaskom merged commit 091f4c0 into master Aug 14, 2021
@mwaskom mwaskom deleted the grid_figure_attribute branch August 14, 2021 20:45
mwaskom added a commit that referenced this pull request Aug 14, 2021
* Add Grid.figure to (eventually) replace Grid.fig

The advantage of this change is that you will be able to do

    obj = sns.<func>()
    obj.figure.<method>

regardless of whether  is figure-level or axes-level.

The `Grid.fig` methods are being soft-deprecated: discouraged in the
attribute documentation but not (currently) issuing a warning when used.

* Update docs, tests, and release notes

(cherry picked from commit 091f4c0)
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