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

Implement Polyphony for piano #35

Open
IsotoxalDev opened this issue Aug 12, 2021 · 11 comments
Open

Implement Polyphony for piano #35

IsotoxalDev opened this issue Aug 12, 2021 · 11 comments
Assignees
Milestone

Comments

@IsotoxalDev
Copy link
Collaborator

Polyphony

The piano instrument as of now can only play one note at a time. It also stops all the music that is playing if stop_note is called on the piano instrument. Playing multiple notes at the same time is important

@Technohacker
Copy link
Member

This may have to be instrument-specific, since certain instruments may be monophonic by design, or have a limited number of voices (pure chiptune)

@IsotoxalDev
Copy link
Collaborator Author

polyphony can be a variable in instrument. the value of polyphony will be the no. of notes that can be played at a time

@razetime
Copy link

if everything is polyphonic, then theres always the option of playing one note at a time. why have an unnecessary extra option?

@Technohacker
Copy link
Member

I've added a method to the Instrument class to let the instrument take an AudioStreamPlayer, with defaults suited for GoDAW. This'll probably be enough, with the instrument deciding how to use those players to play notes

@IsotoxalDev
Copy link
Collaborator Author

if everything is polyphonic, then theres always the option of playing one note at a time. why have an unnecessary extra option?

because we want to cancel out a note if two notes are pressed. completely optional tho.

@IsotoxalDev
Copy link
Collaborator Author

I've added a method to the Instrument class to let the instrument take an AudioStreamPlayer, with defaults suited for GoDAW. This'll probably be enough, with the instrument deciding how to use those players to play notes

so polyphony has to be implemented by the instrument maker?

@Technohacker
Copy link
Member

I've added a method to the Instrument class to let the instrument take an AudioStreamPlayer, with defaults suited for GoDAW. This'll probably be enough, with the instrument deciding how to use those players to play notes

so polyphony has to be implemented by the instrument maker?

Yup, and commit 3680c96 shows one example of this with TripleOsc

@IsotoxalDev
Copy link
Collaborator Author

Nice, Now can this issue be closed?

@Technohacker
Copy link
Member

Not yet, I may need a convenience method for instrument-makers for later :)

@IsotoxalDev
Copy link
Collaborator Author

oh! okay

@IsotoxalDev IsotoxalDev added this to the 0.1.0 milestone Aug 26, 2021
@IsotoxalDev
Copy link
Collaborator Author

So any update on this?

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

No branches or pull requests

3 participants