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 RunInParallel #89

Merged
merged 1 commit into from
Nov 25, 2015

Conversation

EtienneBruines
Copy link
Member

Systemer now includes a function RunInParallel, which indicates whether or not the Entitys should be processed in Parallel.

CollisionSystem only benefits from RunInParallel, if it has more than 40 entities. This is because CollisionSystem is quite fast when there's less than 40 entities, and the overhead of starting goroutines is more.

Each System has its own turning point, which can be defined without much effort (see code).

EtienneBruines added a commit that referenced this pull request Nov 25, 2015
@EtienneBruines EtienneBruines merged commit cdd7e38 into EngoEngine:master Nov 25, 2015
@EtienneBruines
Copy link
Member Author

Merged because the current state of master was just unacceptable (unstable). When rendering 20 objects, 2-4 of them were gone due to concurrency issues. (RenderSystem)

Note that everything is serial (not-concurrent) by default now. This ensures that everything runs the way it should.

@EtienneBruines EtienneBruines mentioned this pull request Nov 25, 2015
@paked
Copy link
Member

paked commented Nov 25, 2015

👍

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

Successfully merging this pull request may close these issues.

2 participants