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 'norm' in init_artists in holoviews/plotting/mpl/raster.py, #6029

Merged
merged 3 commits into from
Dec 12, 2023

Conversation

JulianGiles
Copy link
Contributor

@JulianGiles JulianGiles commented Dec 12, 2023

Added handling of 'norm' kwarg to init_artists in the same way is already present in /holoviews/plotting/mpl/element.py, in order to pass norm when plotting a quadmesh with matplotlib backend. This allows to pass a custom norm and colormap to be able to plot a colobar with discrete, uneven intervals correctly. Related to #5966

Added handling of 'norm' kwarg to init_artists in order to pass norm when plotting with matplotlib backend.
@hoxbro
Copy link
Member

hoxbro commented Dec 12, 2023

Thank you. I can see there is an extra space in the indentation. Can you fix it here (and also if it is there in element.py)?

@hoxbro hoxbro added the type: bug Something isn't correct or isn't working label Dec 12, 2023
@codecov-commenter
Copy link

codecov-commenter commented Dec 12, 2023

Codecov Report

Attention: 4 lines in your changes are missing coverage. Please review.

Comparison is base (2913005) 88.59% compared to head (c11b62c) 88.60%.

Files Patch % Lines
holoviews/plotting/mpl/element.py 0.00% 2 Missing ⚠️
holoviews/plotting/mpl/raster.py 33.33% 2 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #6029   +/-   ##
=======================================
  Coverage   88.59%   88.60%           
=======================================
  Files         315      315           
  Lines       65606    65613    +7     
=======================================
+ Hits        58123    58135   +12     
+ Misses       7483     7478    -5     
Flag Coverage Δ
ui-tests 23.27% <0.00%> (-0.01%) ⬇️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

fixed indentation
fixed indentation
@JulianGiles
Copy link
Contributor Author

Indentation fixed.

@hoxbro
Copy link
Member

hoxbro commented Dec 12, 2023

Thank you.

I will let it merge if the CI passes.

@hoxbro hoxbro enabled auto-merge (squash) December 12, 2023 09:51
@hoxbro hoxbro merged commit 5308bb5 into holoviz:main Dec 12, 2023
11 checks passed
@JulianGiles JulianGiles deleted the patch-1 branch December 12, 2023 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Something isn't correct or isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants