Tcl-bounties icon indicating copy to clipboard operation
Tcl-bounties copied to clipboard

Add "after at" to schedule events for a specific time

Open SpaceManiac opened this issue 7 years ago • 8 comments

A patch has been submitted upstream:

https://core.tcl.tk/tcl/tktview/2953b1b26b58c887435b74923ccef1712ad4b614

SpaceManiac avatar Nov 18 '16 12:11 SpaceManiac

Any chance this could be coordinated with efforts towards http://www.tcl.tk/cgi-bin/tct/tip/302 ?

jima00 avatar Nov 18 '16 18:11 jima00

For the interim, there is a pure-tcl module for tcllib to do roughly this task: http://core.tcl.tk/tcllib/doc/trunk/embedded/www/tcllib/files/modules/cron/cron.html

eviltwinskippy avatar Nov 18 '16 21:11 eviltwinskippy

@jima00 - Yes, I would imagine that we'd want to combine this functionality with 302. My only trepidation is that it changes the semantics of [after] for all the programs that are using something like the 'cron' module to get the desired functionality. [at] really does need to be affected by changes in the system clock.

kennykb avatar Dec 07 '16 22:12 kennykb

It seems like implementing "after at" and then giving some time for programs currently using "after" to schedule wallclock events to switch to "after at" would be a path to making TIP #302 implementable.

So I think this should be submitted as a new TIP with cross-reference to TIP #302.

resuna avatar Apr 07 '17 15:04 resuna

I've provided artificial PR sebres/tcl#4, that fully implements this (together with TIP #302 and many, many other things.

If I get a bit time, and TCT does not see absolute no-go for it (because I've several doubts about), I'll make decent rebase to fossil (with commit-history and review). @flightaware I would like to know your statement also.

sebres avatar Jun 29 '17 19:06 sebres

That looks really impressive. I’ll have to have a sitdown and look it all over. Hopefully it will be possible to convince the TCT.

From: "Serg G. Brester" [email protected] Reply-To: flightaware/Tcl-bounties [email protected] Date: Thursday, June 29, 2017 at 2:56 PM To: flightaware/Tcl-bounties [email protected] Cc: Peter da Silva [email protected], Comment [email protected] Subject: Re: [flightaware/Tcl-bounties] Add "after at" to schedule events for a specific time (#2)

I've provided artificial PR sebres/tcl#4https://github.com/sebres/tcl/pull/4, that fully implements this (together with TIP #302<www.tcl.tk/cgi-bin/tct/tip/302.html> and many, many other things.

If I get a bit time, and TCT does not see absolute no-go for it (because I've several doubts about), I'll make decent rebase to fossil (with commit-history and review). @flightawarehttps://github.com/flightaware I would like to know your statement also.

— You are receiving this because you commented. Reply to this email directly, view it on GitHubhttps://github.com/flightaware/Tcl-bounties/issues/2#issuecomment-312086769, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAF0OoYMRiQVS4gqCZtaNNmFQbLUR5ziks5sJAGDgaJpZM4K2afA.

resuna avatar Jun 29 '17 20:06 resuna

I've extended test-cases a bit, and have provided a performance comparison for above-mentioned branch, if interesting https://github.com/sebres/tcl/pull/4#issuecomment-312252977

sebres avatar Jun 30 '17 13:06 sebres

Reintegrated resp. back-ported as 8.6 based branch in sebres-8-6-event-perf-branch

sebres avatar Jul 10 '17 11:07 sebres