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

Bad cymbal rim dynamics #133

Closed
3hhh opened this issue Dec 17, 2023 · 6 comments
Closed

Bad cymbal rim dynamics #133

3hhh opened this issue Dec 17, 2023 · 6 comments

Comments

@3hhh
Copy link
Contributor

3hhh commented Dec 17, 2023

I noticed that the cymbal rim dynamics are generally bad - there's just one or two velocity levels even though I have rim boost set to zero.

This may be related to my experiments in #126, but I doubt that as the cymbal piezo value should be taken for the MIDI velocity and not the rim switch value.

@corrados
Copy link
Owner

I cannot reproduce this issue. With my Roland CY pad, I get good dynamics when I hit the rim.

@3hhh
Copy link
Contributor Author

3hhh commented Dec 17, 2023 via email

@3hhh
Copy link
Contributor Author

3hhh commented Dec 19, 2023

So I went to some lengths to obtain proper samples from my Teensy with this code.
(Btw it would be useful to have some standardized way of obtaining samples in the repo.)

The results:
crash_rim_hard
crash_rim_med
crash_rim_soft
ride_bell_hard
ride_bell_med
ride_bell_soft
ride_rim_hard
ride_rim_med
ride_rim_soft

Unfortunately I cannot see any fault in my signals as their peak voltage clearly indicates the velocity. Also, the rim switch capacitor circuit correctly indicates that it was used - admittedly sometimes a bit earlier or later than the hit piezo.
So in total I still think that the issue must be with the edrumulus code...

Also, with regards to #126 the plots look neither good for the sample counting of the rim switch open time nor for the second threshold method to discern the bell from the edge. :-(
Maybe the velocity of the hit also has to be taken into account somehow, idk. Or use the hit piezo ripple length, but I don't understand the electronic background for that atm and it makes the different rim input impedances pointless.
Maybe I'll go back to the original circuit...

raw data download

@corrados
Copy link
Owner

Maybe it is an issue with the internal pad trigger parameters. Maybe Edrumulus detects the main peak at the beginning of the burst:
grafik
BTW, you should use logarithmic voltage-axis to better visualize the dynamics of the signal.

@3hhh
Copy link
Contributor Author

3hhh commented Dec 20, 2023 via email

@3hhh
Copy link
Contributor Author

3hhh commented Jan 5, 2024

I went back to the original voltage divider circuit with my parameters and I now have good cymbal rim dynamics for the MPS750x pads again. So apparently this issue was a byproduct of my capacitor circuit for the rim switch.

The Lemon hihat is still not perfect imo, but I'll look into that separately. Maybe the parameters could be chosen a bit differently.

@3hhh 3hhh closed this as completed Jan 5, 2024
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

No branches or pull requests

2 participants