8000 Improved selection widget by blink1073 · Pull Request #3502 · matplotlib/matplotlib · GitHub
[go: up one dir, main page]
Skip to content

Improved selection widget #3502

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.

Sign up for GitHub

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

Merged
merged 18 commits into from
Nov 13, 2014
Merged

Improved selection widget #3502

Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
61ab6df
Factor more behavior into the _SelectorWidget class.
blink1073 Sep 11, 2014
e43c788
Fix indentation
blink1073 Sep 12, 2014
ea732ef
Put onselect in __init__, fix syntax error, make sure lines are anima…
blink1073 Sep 12, 2014
900a6ea
Add tests for the selection widgets
blink1073 Sep 12, 2014
706b5b4
Remove private event handling methods to allow for subclassing old api
blink1073 Sep 12, 2014
ffd715a
Remove draw_rubberband and minor cleanup
blink1073 Sep 12, 2014
53b6056
Add to default_test_modules
blink1073 Oct 19, 2014
ddfcf59
Attempt to fix affine transform error
blink1073 Nov 9, 2014
76721c1
Fix affine transform error
blink1073 Nov 9, 2014
cf66812
Add test cleanup decorator
blink1073 Nov 10, 2014
babc377
Move cleanup to the individual checks
blink1073 Nov 10, 2014
cf58c0a
Fix failing tests
blink1073 Nov 11, 2014
f3261cf
Fix pep8 error
blink1073 Nov 11, 2014
c25efbd
Improve handling of out-of-bounds releases
blink1073 Nov 11, 2014
ce93c98
Revert changes to setupext.py
blink1073 Nov 12, 2014
51c70f0
Style change and use self.blit instead of blit.
blink1073 Nov 12, 2014
d711940
MNT : pep8 + pyflake changes
tacaswell Nov 11, 2014
86f28fa
TST : ensure canvas drawn before testing widets
tacaswell Nov 11, 2014
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Move cleanup to the individual checks
  • Loading branch information
@blink1073
blink1073 committed Nov 12, 2014
commit babc377a5d22a320af30261bc65519fc376cd15b
6 changes: 3 additions & 3 deletions lib/matplotlib/tests/test_widgets.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -63,6 +63,7 @@ def get_event(ax, button=1, xdata=0, ydata=0, key=None, step=1):
return event


@cleanup
def check_rectangle(**kwargs):
ax = plt.gca()
ax.plot([0, 200], [0, 200])
Expand All @@ -87,7 +88,6 @@ def onselect(epress, erelease):
assert ax._got_onselect


@cleanup
def test_rectangle_selector():
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add a @cleanup (from matplotlib.testing.decorators) to this. I suspect that the other failing tests are due the artists form this test leaking out.

check_rectangle()
check_rectangle(drawtype='line', useblit=False)
Expand All @@ -97,6 +97,7 @@ def test_rectangle_selector():
check_rectangle(rectprops=dict(fill=True))


@cleanup
def check_span(*args, **kwargs):
ax = plt.gca()
ax.plot([0, 200], [0, 200])
Expand Down Expand Up @@ -130,13 +131,13 @@ def onmove(vmin, vmax):
assert ax._got_on_move


@cleanup
def test_span_selector():
check_span('horizontal', minspan=10, useblit=True)
check_span('vertical', onmove_callback=True, button=1)
check_span('horizontal', rectprops=dict(fill=True))


@cleanup
def check_lasso_selector(**kwargs):
ax = plt.gca()
ax.plot([0, 200], [0, 200])
Expand All @@ -159,7 +160,6 @@ def onselect(verts):
assert ax._got_onselect


@cleanup
def test_lasso_selector():
check_lasso_selector()
check_lasso_selector(useblit=False, lineprops=dict(color='red'))
Expand Down
0