[issue1676820] Add a PeriodicTimer to threading

[Irit Katriel]

Irit Katriel  added the comment:

I'm closing this as rejected because the comments were not very supportive of 
the idea and there has been no activity for over 10 years.  There are also more 
possibilities for this now with asyncio.

--
resolution:  -> rejected
stage: patch review -> resolved
status: pending -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1676820] Add a PeriodicTimer to threading

[Irit Katriel]

Irit Katriel  added the comment:

Based on the comments, should this be closed as rejected?

--
nosy: +iritkatriel
status: open -> pending

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1676820] Add a PeriodicTimer to threading

[Mark Lawrence]

Changes by Mark Lawrence :


--
nosy:  -BreamoreBoy

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1676820] Add a PeriodicTimer to threading

[Jeffrey Yasskin]

Jeffrey Yasskin  added the comment:

Java's Timer class tends to be discouraged these days in favor of 
ScheduledExecutorService.scheduleAtFixedRate and .scheduleWithFixedDelay 
(http://download.oracle.com/javase/7/docs/api/java/util/concurrent/ScheduledExecutorService.html).
 Could you say why you modeled this interface off of java.util.Timer instead of 
ScheduledExecutor? See http://www.python.org/dev/peps/pep-3148/ for the basis 
of executors in Python.

I'm skeptical of having a PeriodicTimer that embeds a fixed-delay timing. Both 
fixed-rate and fixed-delay are useful for different things, and given the 
ambiguity we probably shouldn't give in to the temptation to guess. 

I'm skeptical of having a new thread per periodic task. It's a little more work 
to multiplex tasks onto shared threads, and maybe that work isn't worth it for 
now, but I think the interface should allow future implementations to multiplex 
their tasks.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1676820] Add a PeriodicTimer to threading

[Antoine Pitrou]

Antoine Pitrou  added the comment:

I don't really agree with "negative periods behave like if the period was 0". 
Negative periods don't mean anything and should just raise ValueError.
(as for 0, well, we can allow it anyway, although it means the timer just 
degenerates into an busy loop)

The doc will have to be converted to the new Sphinx format, as well.

--
nosy: +jyasskin

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1676820] Add a PeriodicTimer to threading

[Éric Araujo]

Changes by Éric Araujo :


--
nosy: +pitrou

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1676820] Add a PeriodicTimer to threading

[Mark Lawrence]

Mark Lawrence  added the comment:

Could someone with commit privileges please take a look at the patch, thanks.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1676820] Add a PeriodicTimer to threading

[Mark Lawrence]

Mark Lawrence  added the comment:

msg52115 "Looks fine to me".  The code change is small, the rest of the patch 
is doc changes or unit tests, could someone run with this?

--
nosy: +BreamoreBoy
versions: +Python 3.2 -Python 2.7, Python 3.1

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1676820] Add a PeriodicTimer to threading

[Daniel Diniz]

Changes by Daniel Diniz :


--
stage:  -> patch review
versions: +Python 2.7, Python 3.1 -Python 2.6, Python 3.0

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1676820] Add a PeriodicTimer to threading

[Christian Heimes]

Changes by Christian Heimes:


--
type:  -> rfe
versions: +Python 2.6, Python 3.0

_
Tracker <[EMAIL PROTECTED]>

_
___
Python-bugs-list mailing list 
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com