Skip to content

Add Async Implementation Reference Module #82

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
Merged
merged 23 commits into from
Jan 10, 2023
Merged

Add Async Implementation Reference Module #82

merged 23 commits into from
Jan 10, 2023

Conversation

Kelwan
Copy link
Contributor

@Kelwan Kelwan commented Nov 21, 2022

Closes #81

@Kelwan Kelwan changed the title Feat/add async ref Add Async Reference Module Nov 21, 2022
@Kelwan Kelwan changed the title Add Async Reference Module Add Async Implementation Reference Module Nov 21, 2022
zaucy
zaucy requested changes Dec 5, 2022
View reviewed changes
@Kelwan Kelwan force-pushed the feat/add-async-ref branch from dfcdff7 to fd78136 Compare December 15, 2022 22:32
zaucy
zaucy reviewed Dec 21, 2022
View reviewed changes
zaucy
zaucy requested changes Dec 21, 2022
View reviewed changes
Kelwan added 17 commits January 10, 2023 13:21
@Kelwan
Initial commit, adding entt stuff
8472ca4
@Kelwan
Set up testing and now working on ref test implementation
f41cb1f
@Kelwan
Added cpp execution, moving into more testing
30dd125
@Kelwan
File restructure, separating code, testing use cases. Going to do a w…
40f6c06 
…orkaround for a BUILD issue invoking certain functions
@Kelwan
Changed tests, more fail safes and made entity creation asynchronous
45b2da7
@Kelwan
Separate functionality, make actually async, and consider more cases
96a21d2
@Kelwan
Moving somewhere else, totally broken
9e40e2b
@Kelwan
Lots of thread safety
0857417
@Kelwan
Removed pending registry, opted for serialization instead. Data probl…
f52720d 
…ems are gone!
@Kelwan
Everything working again, added tick rate options and fixed some bugs
7660f9c
@Kelwan
Split off entity code and ready for review
54e0c9a
@Kelwan
Code review, removing older styles of coding
cf21766
@Kelwan
More code revision
9d4423b
@Kelwan
Done code review
8fbdd85
@Kelwan
Updated tests, rewrote code to fix lifetime issues
7c76a64
@Kelwan
Fixed tick merging issues
2acd5ad
@Kelwan
Fixed tests, ready for review
ff65046
@Kelwan Kelwan force-pushed the feat/add-async-ref branch from 842fd50 to ff65046 Compare January 10, 2023 18:22
@Kelwan Kelwan marked this pull request as ready for review January 10, 2023 18:25
@Kelwan Kelwan merged commit d4069af into main Jan 10, 2023
@Kelwan Kelwan deleted the feat/add-async-ref branch January 10, 2023 19:12
@zaucy zaucy mentioned this pull request Feb 20, 2023
Closed
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zaucy zaucy zaucy approved these changes

Assignees
No one assigned
Labels
None yet
Projects
No open projects
Async API
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Async module reference implementation
2 participants
@Kelwan @zaucy