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

Add a different nbody simulation #81

Merged
merged 2 commits into from
Apr 22, 2021
Merged

Conversation

WhoisDavid
Copy link
Contributor

  • add an n-body simulation of the solar system as 'alternative' n-body
  • fix a typo from previous commit which moved the alternative game of life version.

+ fix a typo from previous commit which moved the alternative game of life version.
@cart
Copy link
Member

cart commented Apr 22, 2021

I think I'd prefer to just allow different implementations of the same thing, rather than categorize the "thing" that came second as "alternate". Would you mind adjusting this to just add a new line like normal (and maybe not right next to the other impl to avoid confusion)?

@WhoisDavid
Copy link
Contributor Author

Yeah sure thing! I just appended mind then.
I was also considering splitting Simulations from Games FYI but unclear if this is relevant at this stage.

Re: Game of Life, I left it as it was before (with alternative) because in this case one is actually a fork of the other.

@cart
Copy link
Member

cart commented Apr 22, 2021

Yeah I think at this point we don't need to distinguish them. Once we migrate to a more interactive awesome-bevy I think it makes sense to start adding more categories.

@cart cart merged commit 4f95f73 into bevyengine:master Apr 22, 2021
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