Skip to content
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

Scheduler: Add event heuristics and performance measuring #193

Open
ghost opened this issue Aug 11, 2020 · 0 comments
Open

Scheduler: Add event heuristics and performance measuring #193

ghost opened this issue Aug 11, 2020 · 0 comments
Assignees
Labels
component: sparta Issue is related to sparta framework enhancement Enhancement or request

Comments

@ghost
Copy link

ghost commented Aug 11, 2020

Along with Scheduler performance improvements (issue #109 and issue #187 ), knowing which events are scheduled the most and which are taking the most time in simulation is very helpful.

To do this (and not be intrusive), the Scheduler shouldn't be touched. Instead, as events are executed, a separate operation will be called that tracks the following:

  1. Event counts (number of times the event was fired)
  2. Event time (average time and total simulation time taken by the event's main handler)

What we can determine from this is overuse/over-scheduling of an Event, and which Events are consuming the most about of time. Naturally, a good profiler can tell the modeler which event is taking the most amount of time, but knowing how many times it was scheduled is eye-opening for a lot of developers.

This separate operation will be a proxy between the Event and its main handler. It will intercept the call from the Scheduler, start tracking, call the event, then stop tracking and return to the Scheduler.

@ghost ghost self-assigned this Aug 11, 2020
@ghost ghost added component: sparta Issue is related to sparta framework enhancement Enhancement or request labels Aug 11, 2020
@ghost ghost changed the title Scheduler: Add event hueristics and performance meauring Scheduler: Add event heuristics and performance meauring Aug 11, 2020
@ghost ghost changed the title Scheduler: Add event heuristics and performance meauring Scheduler: Add event heuristics and performance measuring Aug 12, 2020
@klingaard klingaard assigned klingaard and unassigned ghost Nov 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: sparta Issue is related to sparta framework enhancement Enhancement or request
Projects
None yet
Development

No branches or pull requests

1 participant