modbus reliability and throtling changes#18
Closed
pmp02 wants to merge 3 commits intoAN3Orik:mainfrom
Closed
Conversation
pmp02
commented
Dec 24, 2025
- force close the sockets of the API calls to prevent keep alive and put too much pressure on the IAM module
- split the API call in two calls with different timings (the input registers at 10secs or lower and the configuration registers at 300 seconds - as these don't change that often).
- I also added the Off ventilator status, so I can better monitor the behavior of the unit.
- handling the MB_DISCONNECT and ERROR statuses from the unit, use "temporary unavailable" and back-off API requests exponentially until the unit is available again
- Added several tweaking options in UI directly. These can be used to tune the load on the IAM module.
- max_registers_per_api_request
- web_api_inter_chunk_delay_ms
- status_poll_interval_ms
- config_poll_interval_ms
- status_off_skip_factor: skip api requests N-1 times when the ventilators are off
Author
AN3Orik
added a commit
that referenced
this pull request
Jan 10, 2026
Owner
|
Thanks for pull request, but it's too complicated to merge without break something. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.