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

Require matplotlib 3.6+ for colormaps #2733

Merged
merged 2 commits into from Nov 8, 2023

Conversation

thetorpedodog
Copy link
Contributor

matplotlib.colormaps was introduced in v3.5.0, but scanpy currently specifies only v3.4+. Bump it and everything should be fine.

  • Tests included or not required because: trivial change
  • Release notes not necessary because: trivial change

(this is just my own evaluation of triviality; if you want a release notes entry I am happy to add one)

Copy link

codecov bot commented Nov 6, 2023

Codecov Report

Merging #2733 (45b9e0a) into master (cb31dba) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2733   +/-   ##
=======================================
  Coverage   72.92%   72.92%           
=======================================
  Files         111      111           
  Lines       12134    12134           
=======================================
  Hits         8849     8849           
  Misses       3285     3285           

@thetorpedodog
Copy link
Contributor Author

thetorpedodog commented Nov 6, 2023

See also: conda-forge/scanpy-feedstock#12 [edited to fixed change]

@flying-sheep flying-sheep added this to the 1.9.7 milestone Nov 7, 2023
@flying-sheep
Copy link
Member

@ivirshup I assume you caught that in your minimal-dependencies branch already? Maybe it’s a good time to push it!

`matplotlib.colormaps` was introduced in v3.5.0, but scanpy currently
specifies only v3.4+, and uses other features introduced in 3.6.
Bump it and everything should be fine.
@flying-sheep flying-sheep changed the title Require matplotlib 3.5+ for colormaps Require matplotlib 3.6+ for colormaps Nov 7, 2023
@flying-sheep
Copy link
Member

Do you want to add a release note entry?

@thetorpedodog
Copy link
Contributor Author

Do you want to add a release note entry?

Added. I believe I got the format right.

@Zethson
Copy link
Member

Zethson commented Nov 8, 2023

@flying-sheep @thetorpedodog be aware of #2413

@flying-sheep flying-sheep merged commit d1a2c8f into scverse:master Nov 8, 2023
11 checks passed
meeseeksmachine pushed a commit to meeseeksmachine/scanpy that referenced this pull request Nov 8, 2023
flying-sheep pushed a commit that referenced this pull request Nov 8, 2023
…aps) (#2738)

Co-authored-by: paul fisher <paul@pfish.zone>
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.

None yet

3 participants