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

Add single note post-answer quiz #48

Merged
merged 3 commits into from
Dec 1, 2024
Merged

Add single note post-answer quiz #48

merged 3 commits into from
Dec 1, 2024

Conversation

pganssle
Copy link
Owner

@pganssle pganssle commented Dec 1, 2024

This is one of the methods described in the book for how to train on single notes — after a chord identification, pick one of the three notes from the chord and have the child identify it.

@pganssle pganssle force-pushed the single-note-trainer branch from 22283cc to a5f9759 Compare December 1, 2024 21:46
This is one of the methods detailed in the book for how to start
training on single notes: after an identification, play one of the three
notes that compose the note, and have the child identify it.
@pganssle pganssle force-pushed the single-note-trainer branch from a5f9759 to a56f97d Compare December 1, 2024 21:57
@pganssle pganssle closed this Dec 1, 2024
@pganssle pganssle reopened this Dec 1, 2024
It is not yet clear what the optimal method / UI is going to be for how
to run the single note mode, so I'm erring on the side of giving a lot
of options for how to do it.

The biggest problem with be the transition to the final stage, since
there's no natural "Hey I'm at the next step" level.
This now keeps track of single-note statistics and displays it for the
current session (displaying historical session data is left for a later
PR).
@pganssle pganssle force-pushed the single-note-trainer branch from a56f97d to 72342e2 Compare December 1, 2024 22:17
@pganssle
Copy link
Owner Author

pganssle commented Dec 1, 2024

I tested this as much as I could without pushing to production. I am a bit worried that this might break if someone is in a weird state, but without spending a lot more time on this I don't think I'll find those bugs, so hopefully if this breaks someone they'll report it!

@pganssle pganssle merged commit 2a1e8b5 into main Dec 1, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant