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.

Sign up for GitHub

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

Jump to bottom

Lengths of frequencies (CHG_FREQUENCY and OUT_FREQUENCY) #22

Open

dacwe opened this issue Nov 7, 2024 · 1 comment

dacwe commented Nov 7, 2024

Setting the (defaults) to parameters CHG_FREQUENCY (333kHz) and OUT_FREQUENCY (300kHz) it is read back as something else.

The code says that this is a "integer of length 1": https://github.com/mini-box/ups/blame/8c6db6732181958accb73bc6a405dacf99dde89c/src/lib/HIDOpenUPS.cpp#L73

Is the length (1) correct here? Seems like values above 255 is truncated?

Author

dacwe commented Nov 12, 2024

Super weird behaviour of the OpenUPS when these values are set by this repo. It didn't start properly (no vout) for example.

Had to restore the values using the Windows software to get everything working again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment