Skip to content

Sleep the manager when waiting for next process to run.#6

Open
westonsfiala wants to merge 6 commits into
klavinslab:masterfrom
westonsfiala:master
Open

Sleep the manager when waiting for next process to run.#6
westonsfiala wants to merge 6 commits into
klavinslab:masterfrom
westonsfiala:master

Conversation

@westonsfiala

Copy link
Copy Markdown
Contributor

When running real-time processes, sleeps the manager until the next scheduled process is slated to run.

@klavins klavins self-assigned this Mar 16, 2019
@klavins

klavins commented Mar 19, 2019

Copy link
Copy Markdown
Member

Just FYI, I am holding of on this because some people are expecting different behavior for their projects. Otherwise, I like it.

One thing we might do is add an option to turn on and off the sleep behavior. You could have methods Manager& polite() and Manager& impolite(). I also think an option to just have the manager sleep for some number of milliseconds each time through the loop could be good too. That way we can check for http events at a different frequency, for example.

@westonsfiala

Copy link
Copy Markdown
Contributor Author

With the polite() & impolite() methods we could make the default behavior the exact same as it is now. I would make these changes right now but am prioritizing my project. If I finish up faster than expected, I will make the changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants