-
Notifications
You must be signed in to change notification settings - Fork 4
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 support for spatial audio #26
Conversation
Co-authored-by: Fabian Fritzsche <salzian.dev@gmail.com>
@@ -9,12 +9,18 @@ license = "MIT OR Apache-2.0" | |||
|
|||
[dependencies] | |||
anyhow = "1.0.75" | |||
bevy = { version = "0.11.2", default-features = false } | |||
bevy = { version = "0.11.2", default-features = false, features = ["bevy_audio"] } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should add a test crate that depends on this crate to run unit tests and ensure the crate functions correctly as a library. With just examples, I assume this mistake wouldn't have been obvious.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I haven't done much unit testing before (let alone with Rust) but that sounds reasonable
Cargo.toml
Outdated
bevy_mod_sysfail = "3.0.0" | ||
libfmod = "2.206.2" | ||
once_cell = "1.18.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can't see this being imported anywhere. Where is this used?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lol, no idea where this came from. It's removed now
match query.get_single() { | ||
Ok((velocity, transform)) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch.
See the added example spatial.rs.
Adds components to handle FMOD events in a Bevy way, by implementing AudioSinkPlayback and storing a reference to the FMOD event.
Closes #1
Closes #21
Closes #16