Piano-Trainer icon indicating copy to clipboard operation
Piano-Trainer copied to clipboard

Flat/Sharp notes

Open Finomnis opened this issue 10 years ago • 6 comments

First of all, this is an awesome project and I hope that it is not abandoned yet.

I noticed that it helped me a lot, but I miss generation of flat/sharp notes. Also, I would like an option to change the scale.

So I suggest to add:

  • Possibility to change the base scale
  • Generation of notes outside of the current scale (with lower probability)

Finomnis avatar Feb 28 '16 09:02 Finomnis

Ok I realized from looking at the code that this option already exists. How do you switch it on?

Finomnis avatar Feb 28 '16 09:02 Finomnis

Hi @Finomnis! Thanks for your kind words and suggestions. Actually, I wanted to work on this project again in the next couple of days. In addition to flat/sharp notes and different scales, I also want to add training for rhythm. So, stay tuned!

It will probably take some time, though, since I want to move the project from backbone to react first. Unfortunately, I currently don't have a piano, so I cannot really test things. But maybe you could test some features if they are ready?

philippotto avatar Feb 29 '16 17:02 philippotto

Yes, I do have a piano, and sure, I think I could test things for you. On Feb 29, 2016 18:27, "Philipp Otto" [email protected] wrote:

Hi @Finomnis https://github.com/Finomnis! Thanks for your kind words and suggestions. Actually, I wanted to work on this project again in the next couple of days. In addition to flat/sharp notes and different scales, I also want to add training for rhythm. So, stay tuned!

It will probably take some time, though, since I want to move the project from backbone to react first. Unfortunately, I currently don't have a piano, so I cannot really test things. But maybe you could test some features if they are ready?

— Reply to this email directly or view it on GitHub https://github.com/philippotto/Piano-Trainer/issues/6#issuecomment-190300803 .

Finomnis avatar Feb 29 '16 18:02 Finomnis

Hi Martin, as promised a worked a bit on a new version of the Piano-Trainer. Feel free to test it here: http://philippotto.github.io/Piano-Trainer/es6/

I'd be happy if you could tell me if everything works. The new version is written in ES6 with React instead of coffeescript and backbone. Additionally, it features a setting dialog which allows to adapt the number of notes per chord and whether one wants to have accidentals (flat/sharp notes). I'll lower the probability of flats and sharps and also will add the possibility to have a different base scale, soon.

philippotto avatar Mar 07 '16 15:03 philippotto

Hi!

Cool! Thanks for your work.

I do have a bug report right away, though.

In C-Major scale, which this currently is, it does generate e-sharp and b-sharp, which technically exist, but in a normal scale are represented by f and c. I guess the same accounts for f-b and c-b.

Greetings, Martin

On 07.03.2016 16:47, Philipp Otto wrote:

Hi Martin, as promised a worked a bit on a new version of the Piano-Trainer. Feel free to test it here: http://philippotto.github.io/Piano-Trainer/es6/

I'd be happy if you could tell me if everything works. The new version is written in ES6 with React instead of coffeescript and backbone. Additionally, it features a setting dialog which allows to adapt the number of notes per chord and whether one wants to have accidentals (flat/sharp notes). I'll lower the probability of flats and sharps and also will add the possibility to have a different base scale, soon.

— Reply to this email directly or view it on GitHub https://github.com/philippotto/Piano-Trainer/issues/6#issuecomment-193307364.

Finomnis avatar Mar 07 '16 16:03 Finomnis

Good point! The note generation is very naive at the moment, but I will improve it in combination with the upcoming base scale support. Thanks for pointing that out!

philippotto avatar Mar 07 '16 16:03 philippotto