Skip to content

add interactive colorbar example to gallery#20471

Merged
jklymak merged 5 commits intomatplotlib:masterfrom
richardsheridan:interactive_colorbar
Jul 21, 2021
Merged

add interactive colorbar example to gallery#20471
jklymak merged 5 commits intomatplotlib:masterfrom
richardsheridan:interactive_colorbar

Conversation

@richardsheridan
Copy link
Copy Markdown
Contributor

@richardsheridan richardsheridan commented Jun 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 (and pytest passes).
  • Is Flake 8 compliant (run flake8 on changed files to check).
  • New features are documented, with examples if plot related.
  • Documentation is sphinx and numpydoc compliant (the docs should build without error).
  • Conforms to Matplotlib style conventions (install flake8-docstrings and run flake8 --docstring-convention=all).
  • New features have an entry in doc/users/next_whats_new/ (follow instructions in README.rst there).
  • API changes documented in doc/api/next_api_changes/ (follow instructions in README.rst there).

timhoffm
timhoffm reviewed Jun 20, 2021
View reviewed changes
@timhoffm
Copy link
Copy Markdown
Member

timhoffm commented Jun 20, 2021

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

@richardsheridan richardsheridan mentioned this pull request Jun 22, 2021
Closed
@richardsheridan richardsheridan marked this pull request as draft June 22, 2021 00:51
@richardsheridan
Copy link
Copy Markdown
Contributor Author

richardsheridan commented Jun 22, 2021

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

@richardsheridan
Copy link
Copy Markdown
Contributor Author

richardsheridan commented Jun 22, 2021

Also something about the new ColorbarAxes 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.

@richardsheridan richardsheridan marked this pull request as ready for review July 8, 2021 14:20
@richardsheridan
Copy link
Copy Markdown
Contributor Author

richardsheridan commented Jul 8, 2021

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

@timhoffm
Copy link
Copy Markdown
Member

timhoffm commented Jul 8, 2021

flake8:

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

@jklymak
Copy link
Copy Markdown
Member

jklymak commented Jul 21, 2021

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?

@jklymak jklymak merged commit 4fdff43 into matplotlib:master Jul 21, 2021
@jklymak
Copy link
Copy Markdown
Member

jklymak commented Jul 21, 2021

Thanks!

@richardsheridan richardsheridan deleted the interactive_colorbar branch July 21, 2021 02:00
@QuLogic QuLogic added this to the v3.5.0 milestone Jul 21, 2021
@richardsheridan richardsheridan mentioned this pull request Sep 27, 2021
Merged
7 tasks
@greglucas greglucas mentioned this pull request Sep 30, 2021
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@timhoffm timhoffm timhoffm left review comments

@jklymak jklymak jklymak approved these changes

Assignees

No one assigned

Labels

topic: color/colorbar

Projects

None yet

Milestone

v3.5.0

Development

Successfully merging this pull request may close these issues.

4 participants

@richardsheridan @timhoffm @jklymak @QuLogic