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

Update network cache building to use execution buckets #101

Closed
S-S-X opened this issue Oct 13, 2020 · 0 comments · Fixed by #136
Closed

Update network cache building to use execution buckets #101

S-S-X opened this issue Oct 13, 2020 · 0 comments · Fixed by #136
Assignees
Labels
Enhancement New feature or request

Comments

@S-S-X
Copy link
Member

S-S-X commented Oct 13, 2020

Upgrade network building so that it can be used to build caches for any network size and change technic.switch_max_range to be absolute count of network nodes to be checked and added to cache.

After technic.switch_max_range is reached mark network as incomplete and save queues, next globalstep then continues where previous one stopped. Mark network as complete after there's no queue available.

Normal network execution should then begin if network is marked as complete and if not then continue building network.

Multiple switching stations might create challenges here as network building can start at different locations, not sure how to handle this. One way could be to allow only single network build operation and only start new ones after one under construction is completed or disappears.

Also consider possibility to do same for actual network execution for huge and/or laggy networks.

Depends on #96 and should be started only after that PR is merged.

@S-S-X S-S-X added Enhancement New feature or request Blocked Has unmerged dependency or some other blockage. Check details before working on this. labels Oct 13, 2020
@S-S-X S-S-X removed the Blocked Has unmerged dependency or some other blockage. Check details before working on this. label Nov 1, 2020
@S-S-X S-S-X self-assigned this Dec 7, 2020
@S-S-X S-S-X closed this as completed in #136 Dec 9, 2020
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