-
Notifications
You must be signed in to change notification settings - Fork 548
A port of an early version of Baystation's new instruments #37447
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
Conversation
first commit for instruments, should have everything ready
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.
Still working through this but aside from the controller issue I'm mainly just seeing style issues. General notes:
- Avoid same-line if's/for's/etc. The improved readability of splitting these into multiple lines is worth the added linecount.
- src.*** is being applied liberally here - you typically do not need to put src. before accessing/setting a var within scope of the var's owner. No need to correct the existing instances; it's just worth noting for the future.
The PR that saved /vg/station |
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.
Looking good, I only noticed one strange check and minor infractions of code readability. Not bad for a port. I'm still a little blind in the music math parts of the code. They seem to work correctly...
What this does
I ported an early version of Baystation's new instruments, for now there's only one new physical instruments (the synth) which contains all of them. Future PRs will add more physical instruments (for now, old instruments are still there in the code and still work). The new interface looks like this.

New features include:
Why it's good
Because our instruments sound bad and work bad.
How it was tested
I tested all the new instruments that come with the synthesizer and mostly do sound good. I didn't notice any performance drop while playing. Wrenching and unwrenching the synth works. I'm going to upload some videos of it as well. (audio only)
campanella real.webm
Sprites
I have three different sprites for the synth, made by Blonichu, Burntdevil and Turnip. The synth will pick a random sprite when created.



Changelog
🆑