-
Notifications
You must be signed in to change notification settings - Fork 53
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
Branch/no model tick #473
Open
scottnc27603
wants to merge
21
commits into
development
Choose a base branch
from
branch/no-model-tick
base: development
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Branch/no model tick #473
Commits on Feb 23, 2015
-
Handle reflector actions as they arrive, without waiting for a tick.
There isn't any reason to delay execution. `kernel.tick` is still called exactly once per reflector tick. Also, don't bother advancing time after the last authorized action. Nothing executes at that time if it is between actions, so nothing will see it. This is in preparation for keeping tick messages out of the queue.
Configuration menu - View commit details
-
Copy full SHA for aac8b56 - Browse repository at this point
Copy the full SHA aac8b56View commit details -
Don't record ticks in the queue. Don't tick nodes and model drivers.
If the model sees individual tick messages, then unexecuted ticks are part of the application state. All ticks must pass through the message queue, and unexecuted ticks must be included in the replication and persistence data. This adds unnecessary bulk to the state data and can increase buffer delays during poor network conditions. It also prevents the reflector from ticking clients on slower networks at a slower rate. This commit interprets reflector ticks only as authorization to move kernel time forward. Ticks are no longer placed in the queue. `model.ticking` is no longer called on model drivers, and `node.tick` is no longer called for nodes providing a `tick` function. `view.ticked` is still called for view drivers since views don't affect the application state. View ticks are still called exactly once per reflector tick message. Nodes and model drivers that make use of tick messages will need to be updated. If necessary, the kernel may be able to insert `future` calls to provide backward compatability.
Configuration menu - View commit details
-
Copy full SHA for 41229c1 - Browse repository at this point
Copy the full SHA 41229c1View commit details -
Configuration menu - View commit details
-
Copy full SHA for e5898aa - Browse repository at this point
Copy the full SHA e5898aaView commit details -
Configuration menu - View commit details
-
Copy full SHA for ed2b3c1 - Browse repository at this point
Copy the full SHA ed2b3c1View commit details -
Configuration menu - View commit details
-
Copy full SHA for a1b5e5d - Browse repository at this point
Copy the full SHA a1b5e5dView commit details -
Configuration menu - View commit details
-
Copy full SHA for 65d401b - Browse repository at this point
Copy the full SHA 65d401bView commit details -
Configuration menu - View commit details
-
Copy full SHA for 57b03ec - Browse repository at this point
Copy the full SHA 57b03ecView commit details -
Configuration menu - View commit details
-
Copy full SHA for 563d7b8 - Browse repository at this point
Copy the full SHA 563d7b8View commit details -
Configuration menu - View commit details
-
Copy full SHA for 3c0f3cb - Browse repository at this point
Copy the full SHA 3c0f3cbView commit details -
Configuration menu - View commit details
-
Copy full SHA for b07308c - Browse repository at this point
Copy the full SHA b07308cView commit details -
Configuration menu - View commit details
-
Copy full SHA for bf0b2ee - Browse repository at this point
Copy the full SHA bf0b2eeView commit details -
Configuration menu - View commit details
-
Copy full SHA for b0ee31b - Browse repository at this point
Copy the full SHA b0ee31bView commit details -
Configuration menu - View commit details
-
Copy full SHA for c159037 - Browse repository at this point
Copy the full SHA c159037View commit details -
Configuration menu - View commit details
-
Copy full SHA for cd1ab09 - Browse repository at this point
Copy the full SHA cd1ab09View commit details -
Configuration menu - View commit details
-
Copy full SHA for 3b683c0 - Browse repository at this point
Copy the full SHA 3b683c0View commit details -
Configuration menu - View commit details
-
Copy full SHA for 4e0c26c - Browse repository at this point
Copy the full SHA 4e0c26cView commit details -
Configuration menu - View commit details
-
Copy full SHA for 06b34ee - Browse repository at this point
Copy the full SHA 06b34eeView commit details -
Configuration menu - View commit details
-
Copy full SHA for 874f5b6 - Browse repository at this point
Copy the full SHA 874f5b6View commit details -
Configuration menu - View commit details
-
Copy full SHA for 58b7769 - Browse repository at this point
Copy the full SHA 58b7769View commit details -
Configuration menu - View commit details
-
Copy full SHA for a16b240 - Browse repository at this point
Copy the full SHA a16b240View commit details -
Configuration menu - View commit details
-
Copy full SHA for aadaea2 - Browse repository at this point
Copy the full SHA aadaea2View commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.