-
Notifications
You must be signed in to change notification settings - Fork 79
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
Calibration update #379
base: main
Are you sure you want to change the base?
Calibration update #379
Conversation
…O pin wrappers from europi.py instead of re-declaring them.
…n (5V voltage sources seem way more common in EuroRack modules than 10V). Move diagnistic out of contrib, into new firmware/tools namespace. Put calibration here too. Update README, programming instructions, menu accordingly. Add calibration.md with more detailed description of calibration process
… necessary exception handling for Pico 2's (currently) unsupported temperature sensor
Did some testing. Found yet another annoying inconsistency with the RP2350 that's been addressed (pin 24 doesn't appear to indicate if the USB is connected or not). Added a new class similar to the Low-accuracy, 5V calibration is, as-expected, noisier, but good-enough for quick calibration. Working on fixing up the tests; looks like |
Tests appear to be passing now; I think it's ready for review. |
Added per-output calibration instead of just calibrating CV1. Updated description accordingly. New calibration creates
In the case of someone using the older CV1-only calibration, Sample
The |
Several changes related to calibration:
firmware/tools
namespaceeuropi.py
is modified to work with both the new & old calibrations, so anyone who's calibrated using only the CV1-only method will still have usable calibration.Thermometer
class toeuropi.py
to act as a wrapper for the temperature sensor on pin4. Add exception-handling needed for the Pico 2 to prevent crashes when trying to read the temperature sensor.