Skip to content

Conversation

siemdejong
Copy link
Contributor

@siemdejong siemdejong commented Aug 12, 2024

This PR proposes to change all matplotlib.axes.Subplot typehints and isinstance checks with matplotlib.axes.Axes to support newer matplotlib versions. This would enable users to e.g. distribute their packages using dabest with specific style files.

Fixes #183

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@Jacobluke-
Copy link
Collaborator

Hi @siemdejong , you will need to specify a more detailed version number for matplotlib for it to run.

@siemdejong
Copy link
Contributor Author

Hey @Jacobluke-, thanks for considering this PR and reviewing. Does 3164aee satisfy your request?

@Jacobluke-
Copy link
Collaborator

Hi @siemdejong, our Python workflow test the package with the minimum supported version of Python (3.8). And matplotlib 3.8 only support Python >= 3.9 hence the workflow didn't run through. You can install pytest-mpl plugin and run pytest on the root directory on your local environment to see if all the tests pass.

@siemdejong
Copy link
Contributor Author

Python 3.8 will reach end of life this October [1]. Are you planning to keep python 3.8 support after October?

[1] https://devguide.python.org/versions/

@Jacobluke-
Copy link
Collaborator

Thank you for your information, we will discuss about it in the team and likely to push commits to your PR to address this issue.

@Jacobluke- Jacobluke- changed the base branch from master to vnbdev September 17, 2024 07:22
@Jacobluke- Jacobluke- requested a review from JAnns98 September 23, 2024 06:59
…uests/siemdejong/issue-183

# Conflicts:
#	dabest/_modidx.py
#	dabest/misc_tools.py
#	dabest/plot_tools.py
#	dabest/plotter.py
#	nbs/API/misc_tools.ipynb
#	nbs/API/plot_tools.ipynb
#	nbs/API/plotter.ipynb
Copy link
Collaborator

@JAnns98 JAnns98 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Everything looks in order! Happy to continue merging

@Jacobluke- Jacobluke- merged commit d3d531f into ACCLAB:vnbdev Sep 25, 2024
2 checks passed
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.

Support newer versions of matplotlib (>=3.8)

3 participants