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.

Sign up for GitHub

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

Jump to bottom

Sync-Point ergonomics #45

Closed
5 tasks done
minecrawler opened this issue Jan 14, 2022 · 1 comment
Closed
5 tasks done

Sync-Point ergonomics #45

minecrawler opened this issue Jan 14, 2022 · 1 comment
Assignees
@minecrawler
Labels
enhancement New feature or request
Projects
New Features
Milestone
0.5.1

Comments

@minecrawler
Copy link
Collaborator

minecrawler commented Jan 14, 2022 •
edited

Sync-Points should be

  • easy to create and extend (with other sync-points) 50c66d9 2839b08
  • able to have labels, which allow arbitrary insertion of systems before or after them af4e9bb
  • serializable as prefabs, so a complex pipeline can be created separately, stored and loaded and have a very simple syntax 50c66d9
  • safe to use - we need checks for loops! bceb421
  • execute stuff on point (commands.flush(), world.maintain(), custom logic like "extract to other thread") bb7ecae
@minecrawler minecrawler added the enhancement New feature or request label Jan 14, 2022
@minecrawler minecrawler added this to the 0.5.1 milestone Jan 14, 2022
@minecrawler minecrawler added this to To do in New Features via automation Jan 14, 2022
minecrawler added a commit that referenced this issue Apr 27, 2022
@minecrawler
#45 implement scheduling and sync-point creation prefabs
50c66d9
@minecrawler minecrawler self-assigned this Apr 27, 2022
minecrawler added a commit that referenced this issue Apr 27, 2022
@minecrawler
#45 implement sync handlers
bb7ecae
minecrawler added a commit that referenced this issue Jun 12, 2022
@minecrawler
#45 implement sync-point labels and global registration
af4e9bb
minecrawler added a commit that referenced this issue Jun 12, 2022
@minecrawler
#45 add sync-point updater for convenience
2839b08 
useful to update the name, too!
minecrawler added a commit that referenced this issue Jun 12, 2022
@minecrawler
#45 implement loop checks for sync-points
bceb421
@minecrawler
Copy link
Collaborator Author

upon investigation, it is not really possible to make a good API to insert systems around sync-point because systems are part of stages. As such, that part will not be implemented and it remains with the user to create good pipelines (which is best practice anyway)

New Features automation moved this from To do to Done Jun 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@minecrawler minecrawler

Labels
enhancement New feature or request
Projects
New Features
Status: Done
New Features
Done
Milestone
0.5.1
Development

No branches or pull requests
1 participant
@minecrawler