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

Remove smooth-bevy-cameras dependency from spatial example #61

Merged
merged 8 commits into from
Nov 23, 2023

Conversation

GitGhillie
Copy link
Collaborator

This will make it easier to update bevy_fmod for new releases of Bevy.
Took some inspiration from the Bevy spatial example.
Feedback welcome!

@GitGhillie GitGhillie added the dependencies Pull requests that update a dependency file label Oct 21, 2023
Salzian
Salzian previously approved these changes Oct 21, 2023
Copy link
Owner

@Salzian Salzian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice simplification. Small improvement idea: Spawn the camera further back so the Doppler effect is immediately present.

@GitGhillie
Copy link
Collaborator Author

Assuming you mean the spatial effect; I've now adjusted it so the cube is in view the whole time. Let me know what you think, I think it's better

@GitGhillie GitGhillie mentioned this pull request Oct 28, 2023
3 tasks
@Salzian
Copy link
Owner

Salzian commented Nov 23, 2023

I guess this becomes obsolete with #60 ? Then we can close this PR.

@GitGhillie
Copy link
Collaborator Author

IMO it's still a good move, we'll run into the same issue when preparing for 0.13, 0.14, etc.
Up to you

@GitGhillie GitGhillie merged commit f282e49 into main Nov 23, 2023
@GitGhillie GitGhillie deleted the remove-fly-cam branch November 23, 2023 20:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants