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

IPS - Multi-IPS-Instance Setup + Syncing #11

Closed
Tmktahu opened this issue Oct 5, 2021 · 1 comment · Fixed by #12
Closed

IPS - Multi-IPS-Instance Setup + Syncing #11

Tmktahu opened this issue Oct 5, 2021 · 1 comment · Fixed by #12
Assignees
Labels
Enhancement New feature or request

Comments

@Tmktahu
Copy link
Owner

Tmktahu commented Oct 5, 2021

One way we can achieve a ~0.2 second update time is to run multiple instances of IPS at the same time offset from each other. The code is compatible with itself, so you can already run several instances at once without problems.

But because line execution time is not perfectly 0.2 seconds, multiple instances will eventually become out of sync with each other. To handle this we want to try and add a line to the execution code that automatically syncs different IPS chips together after some time has passed.

Because this would add 0.2 seconds to update time, bringing us to 0.8 seconds. Because of that we want to make this multi-instance IPS version separate from the 1-chip version. In addition, it will need to run on 4 chips and the syncing logic will need to handle starting each chip on a different line to keep them offset from each other.

@Tmktahu Tmktahu added the Enhancement New feature or request label Oct 5, 2021
@Tmktahu Tmktahu self-assigned this Oct 5, 2021
@Tmktahu Tmktahu linked a pull request Nov 1, 2021 that will close this issue
@Tmktahu
Copy link
Owner Author

Tmktahu commented Nov 1, 2021

#12 introduces IPSA which implements a new version of IPS that runs async. I'm going to close this ticket with the completion of IPSA and leave black-magic discussions for other places.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant