8000 TextBox: Added set_val method for convenient scripted setting and updating. by smithsp · Pull Request #1 · HastingsGreer/matplotlib · GitHub
[go: up one dir, main page]

Skip to content

TextBox: Added set_val method for convenient scripted setting and updating. #1

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

Closed
wants to merge 1,800 commits into from

Conversation

smithsp
Copy link
@smithsp smithsp commented Jun 17, 2016

@QuadmasterXLII
Thanks for adding the TextBox as a Pull Request to matplotlib (matplotlib#5375). I took what you added, and added a set_val method. Let me know if I need to change anything for you to accept this into your branch and thus to go into matplotlib#5375.
@smithsp

@smithsp
Copy link
Author
smithsp commented Jun 21, 2016

@tacaswell Can you tell why the travis build would have failed for this branch/pull request, but not for matplotlib#5375?

@tacaswell
Copy link

There may be fixes on the master branch which are not included on this branch. I can not tell much else with out digging deeper.

@smithsp
Copy link
Author
smithsp commented Jun 21, 2016

@tacaswell Thanks for taking a look.

mlub and others added 27 commits July 18, 2016 18:37
These are fixes for all issues found in axes_grid, color, and
event_handling sections on the examples page.

axisartist/demo_curvelinear_grid.py - Set linewidth=2.0 and grid zorder=0 so
the plotted lines actually visible

event_handling/keypress_demo.py - added usage instruction as axes title

event_handling/lasso_demo.py - added usage instruction as axes title

event_handling/looking_glass.py - added usage instruction as axes title

event_handling/resample.py - removed gtk and hardcoded wave file dependencies
and just created a sample signal within the script

event_handling/trifinder_event_demo.py - zip(l1,l2) call changed to
list(zip(l1,l2)) to be python3 compatible

event_handling/viewlims.py - The focus box that is supposed to appear in the
left axes when user zooms in the right axes wasn't visible. Set linewidth=1.0
`annotate(..., arrowprops={})` now uses an arrow with the default arrow
properties, rather than no arrow.  Note that this matches the docstring
of `annotate`, which indicates that "arrowprops, if not None, is a
dictionary of line properties (see matplotlib.lines.Line2D) for the
arrow that connects annotation to the point."

Comes down to checking for `None` instead of falsiness of `arrowprops`.
DOC: Added contribute information to readme
Apparently unused and with no effect ever since they were added.
The official xkcd font https://github.com/ipython/xkcd-font
was missing from the list of what is looked for when the
user selects xkcd mode. Since it is the "official" font, it
has been added to the top of the list.
…nect-dragging-callbacks

FIX: Don't forget to disconnect callbacks for dragging.
While nose is pretty happy with old ones, this has to be done
to conform with other tests and default pytest search pattern.
ENH: Add xkcd font as one of the options in xkcd style
ENH: Add xkcd font as one of the options in xkcd style
f0k and others added 4 commits August 26, 2016 11:19
…ons-picker

MNT: Don't warn in Collections.contains if picker is not numlike.
…tive-invalid-log

FIX: Catch invalid interactive switch to log scale.
ENH: PGF Backend: Support interpolation='none'
@smithsp smithsp force-pushed the text-box-widget branch 2 times, most recently from 8370ddd to 257b776 Compare August 29, 2016 16:49
@smithsp
Copy link
Author
smithsp commented Oct 6, 2016

I am closing this because it got merged into the main matplotlib repo through a different pull request.

@smithsp smithsp closed this Oct 6, 2016
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.

0