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

Disable periodic reports not preserved between settings save/read #310

Open
ncolomer opened this issue Sep 19, 2024 · 0 comments
Open

Disable periodic reports not preserved between settings save/read #310

ncolomer opened this issue Sep 19, 2024 · 0 comments

Comments

@ncolomer
Copy link

Hello,

The "Disable periodic On/Off reports for GPIO outputs" option in the "Expert" tab is not preserved between settings save and read.

Steps to reproduce:

  • run FirmwareConfig.exe
  • tick "Disable periodic On/Off reports for GPIO outputs" checkbox in "Expert" tab
  • click "Save settings to a file" and save as test.ini
  • click File / Reset to defaults (optional, the issue raises anyway)
  • click File / Read settings from a file and load test.ini

The option will be shown as unchecked in the "Expert" tab. Additionally, I don't see any mention of the option in the produced ini file. This may be the cause of the problem (not written at save time).

ptvoinfo pushed a commit that referenced this issue Sep 30, 2024
[-][All chips] Reporting of ADC value (raw) was fixed.
[-][All chips] Manual reading of the UART sensor analog value did not work (#309).
[-][All chips] The firmware no longer returns an error when the higher-level system tries to set the periodic report interval while this function is disabled in the configuration. It simply ignores it quietly.
[+][All chips] Added binding capability to a group when the PTVO device is the command source (#156).
[*][CC2530] HC-SR04 sensor is now unavailable in the router mode.
[-][Configurator] The "Disable periodic On/Off reports" option was not saved  (#310).
[-][Configurator] Configuration for outputs 8-16 was not loaded correctly from the file.
[*][Configurator] Other minor improvements.
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

1 participant