Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

add interactive colorbar example to gallery#20471

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged

Conversation

richardsheridan
Copy link
Contributor

@richardsheridanrichardsheridan commentedJun 19, 2021
edited
Loading

PR Summary

Added an example to the gallery for discussion regarding#19515. All that needs to happen in each event is to modify the colorbar norm vmin and vmax, although the particular interactivity that was desired in#19515 would require a little more creativity in the callbacks.

Hoping for feedback as to:

  1. whether to actually include in gallery
  2. style of new gallery page
  3. code review

PR Checklist

  • Has pytest style unit tests (andpytest passes).
  • IsFlake 8 compliant (runflake8 on changed files to check).
  • New features are documented, with examples if plot related.
  • Documentation is sphinx and numpydoc compliant (the docs shouldbuild without error).
  • Conforms to Matplotlib style conventions (installflake8-docstrings and runflake8 --docstring-convention=all).
  • New features have an entry indoc/users/next_whats_new/ (follow instructions in README.rst there).
  • API changes documented indoc/api/next_api_changes/ (follow instructions in README.rst there).

@timhoffm
Copy link
Member

BTW thanks for taking the time to put this example together!

@richardsheridan
Copy link
ContributorAuthor

I feel#20479 blocks this PR so marking as draft until resolved.

@richardsheridan
Copy link
ContributorAuthor

Also something about the newColorbarAxes detaches the mouse event xdata and ydata from the data range, so this only coincidentally worked since the range of the data was 0..1, another blocker.

@richardsheridanrichardsheridan marked this pull request as ready for reviewJuly 8, 2021 14:20
@richardsheridan
Copy link
ContributorAuthor

Unblocked by#20501, thanks for everyone's help there! (Somehow I didn't get any notifications about that PR?)

@timhoffm
Copy link
Member

flake8:

./examples/userdemo/colormap_interactive_adjustment.py:51:80: E501 line too long (87 > 79 characters)

@jklymak
Copy link
Member

Maybe I'm doing something wrong, but this only works for one click for me and then the colorbar stops responding. Is that expected, or am I doing something wrong?

@jklymakjklymak merged commit4fdff43 intomatplotlib:masterJul 21, 2021
@jklymak
Copy link
Member

Thanks!

@richardsheridanrichardsheridan deleted the interactive_colorbar branchJuly 21, 2021 02:00
@QuLogicQuLogic added this to thev3.5.0 milestoneJul 21, 2021
@richardsheridanrichardsheridan mentioned this pull requestSep 27, 2021
7 tasks
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@timhoffmtimhoffmtimhoffm left review comments

@jklymakjklymakjklymak approved these changes

Assignees
No one assigned
Projects
None yet
Milestone
v3.5.0
Development

Successfully merging this pull request may close these issues.

4 participants
@richardsheridan@timhoffm@jklymak@QuLogic

[8]ページ先頭

©2009-2025 Movatter.jp