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

c0 causes a note to play forever, is it working as intended? #398

Closed
xcoderreal opened this issue Sep 11, 2021 · 2 comments
Closed

c0 causes a note to play forever, is it working as intended? #398

xcoderreal opened this issue Sep 11, 2021 · 2 comments
Labels

Comments

@xcoderreal
Copy link

🐞 Bug report 🐞

Description

c0 causes a note to play forever

Steps to Reproduce

piano: c4 c4 c0

Yes

Expected Behavior

Seems like it should have been an error or forbidden?

(I haven't dug into the code but instead just skimmed through the Notes documentation.)

Actual Behavior

Environment

Operating system and version:

Alda version:

$ alda version
alda 2.0.5
$ alda-player info
alda-player 2.0.5
log path: [skipped here]

Health check:

$ alda doctor
OK  Parse source code
OK  Generate score model
OK  Ensure that there are no stale player processes
OK  Find an open port
OK  Send and receive OSC messages
OK  Locate alda-player executable on PATH
OK  Check alda-player version
OK  Spawn a player process
OK  Ping player process
OK  Play score
OK  Export score as MIDI
OK  Locate player logs
OK  Player logs show the ping was received
OK  Shut down player process
OK  Spawn a player on an unknown port
OK  Discover the player
OK  Ping the player
OK  Shut the player down
OK  Start a REPL server
nREPL server started on port 51860 on host localhost - nrepl://localhost:51860
OK  Find the REPL server
OK  Interact with the REPL server
OK  Shut down the REPL server

Logs:

@xcoderreal xcoderreal added the bug label Sep 11, 2021
@daveyarwood
Copy link
Member

@xcoderreal Thanks for the issue! You have a good point. I think a note length of 0 should probably be treated as an error. I'll leave this open for a while in case anyone else has any thoughts they'd like to share, but if there are no objections, I think I will make it so that e.g. c0 results in an error.

@daveyarwood
Copy link
Member

Fixed in Alda 2.0.6 🎉

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

No branches or pull requests

2 participants