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

Added runtime_get current_tick #823

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

Jujstme
Copy link
Contributor

@Jujstme Jujstme commented Jul 8, 2024

This adds a new tick field that increases by 1 at every call of the update function. This can be used for different purposes, eg. for caching data or to check (together with the current tick rate, albeit in an approximate way), the amount of time passed when executing code between iterations of the main autosplitting loop.

This adds a new tick field that increases by 1 at every call of the update function. This can be used for different purposes, as in for caching data or to check (together with the current tick rate, albeit in an approximate way), the amount of time passed when executing code between iterations of the main autosplitting loop.
@CryZe
Copy link
Collaborator

CryZe commented Aug 22, 2024

I'm not sure I like it. This adds an atomic increment (slower than a normal increment even) to every update tick, even if the auto splitter never even needs this functionality. An auto splitter can easily just increment their own counter.

@CryZe CryZe added needs further discussion It is unclear how to progress without making further decisions. auto splitting This is about the auto splitting implementation. labels Aug 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto splitting This is about the auto splitting implementation. needs further discussion It is unclear how to progress without making further decisions.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants