NotificationsYou must be signed in to change notification settings
Fork7.9k
Star21.3k

Commitf6e282b

committed

FIX: Event loop timers should only run for the specified time

The implementation of start_event_loop would previously just countthe number of sleeps that occurred. But this could lead to longerevent loop times if flush_events() added time into the loop. Wewant the condition to be dependent on the end-time so we don'trun our loop longer than necessary.

1 parent0f6a157 commitf6e282bCopy full SHA for f6e282b

File tree

1 file changed

-4

lines changed

lib/matplotlib
- backend_bases.py

1 file changed

-4

lines changed

`‎lib/matplotlib/backend_bases.py`

Lines changed: 2 additions & 4 deletions

Original file line number	Diff line number	Diff line change
`@@ -2321,12 +2321,10 @@ def start_event_loop(self, timeout=0):`
`2321`	`2321`	`iftimeout<=0:`
`2322`	`2322`	`timeout=np.inf`
`2323`	`2323`	`timestep=0.01`
`2324`		`-counter=0`
`2325`		`-self._looping=True`
`2326`		`-whileself._loopingandcounter*timestep<timeout:`
	`2324`	`+t_end=time.time()+timeout`
	`2325`	`+whiletime.time()<t_end:`
`2327`	`2326`	`self.flush_events()`
`2328`	`2327`	`time.sleep(timestep)`
`2329`		`-counter+=1`
`2330`	`2328`
`2331`	`2329`	`defstop_event_loop(self):`
`2332`	`2330`	`"""`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

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

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

Commitf6e282b

File tree

1 file changed

1 file changed

`‎lib/matplotlib/backend_bases.py`

0 commit comments