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

Yarns INIT resets calibration settings #19

Closed
dturner opened this issue Mar 16, 2019 · 3 comments
Closed

Yarns INIT resets calibration settings #19

dturner opened this issue Mar 16, 2019 · 3 comments

Comments

@dturner
Copy link

dturner commented Mar 16, 2019

Summary: On Yarns if INIT is chosen the calibration settings are lost.

Steps to reproduce:

  • Connect a multimeter to Yarns CV1 output to measure the voltage
  • Calibrate Yarns CV1 0V voltage (this doesn't need to be exactly 0V, just some different value than is currently shown on the multimeter).
  • Save calibration settings
  • Choose INIT (to get a default program)
  • Send a C1 note to Yarns
  • Check the value shown on the multimeter

Expected result

  • Multimeter shows the calibrated value which we just set

Actual result

  • Multimeter shows the original value before calibration

Note that the calibration settings can be saved by saving a program after performing calibration. For example, if you save to P1 then the calibration settings will be restored on boot. It's just using INIT which wipes them.

It may be that this is as designed, however, I wanted to check as it seems like resetting the calibration just to get a default program (using INIT) is not expected behaviour.

@dturner
Copy link
Author

dturner commented Mar 16, 2019

Full disclosure, this is a DIY build. Please close if you think there's no likelihood of it affecting other devices (and sorry for the noise).

@pichenettes
Copy link
Owner

Thanks for the bug report! This is fixed.

@dturner
Copy link
Author

dturner commented Mar 18, 2019

Wow. Thank you so much émilie. Tested and fixed :D

synthetiv pushed a commit to synthetiv/eurorack that referenced this issue Sep 29, 2021
Fixes pichenettes#19
Also make attenuation reset correctly, fixing bug that creeped in in the
fix to pichenettes#15
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