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

Pylbo: improved subplot addition #77

Merged
merged 5 commits into from
Sep 8, 2021
Merged

Pylbo: improved subplot addition #77

merged 5 commits into from
Sep 8, 2021

Conversation

n-claes
Copy link
Owner

@n-claes n-claes commented Sep 8, 2021

PR description

Improves the _add_subplot_axes method in the FigureManager to be compatible with more complex subplot layouts, also fixes the deprecationwarning for change_geometry from matplotlib 3.4 onwards. Closes #76.
Also fixes the "packaging" dependency issue in the setup.py installation file.

@n-claes n-claes added bug Something isn't working pylbo Affects the post-processing framework Pylbo labels Sep 8, 2021
@n-claes n-claes added this to the v1.1 milestone Sep 8, 2021
@codecov
Copy link

codecov bot commented Sep 8, 2021

Codecov Report

Merging #77 (b0ad1d3) into develop (9beff37) will increase coverage by 0.11%.
The diff coverage is 91.17%.

@@             Coverage Diff             @@
##           develop      #77      +/-   ##
===========================================
+ Coverage    86.88%   87.00%   +0.11%     
===========================================
  Files          102      102              
  Lines         5885     5885              
===========================================
+ Hits          5113     5120       +7     
+ Misses         772      765       -7     
Flag Coverage Δ
legolas 88.74% <ø> (ø)
pylbo 83.50% <91.17%> (+0.35%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

@n-claes n-claes merged commit 775e11b into develop Sep 8, 2021
@n-claes n-claes deleted the bugfix/mpl_geometry branch September 8, 2021 14:17
@n-claes n-claes mentioned this pull request Oct 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working pylbo Affects the post-processing framework Pylbo
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant