Repository navigation
Show leak rates immediately and add independent long term measurements - #4
Merged
Merged
Conversation
SebRoLENS
marked this pull request as ready for review
October 5, 2026 09:03
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
1 participant
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.
Pressure-loss rates and warnings previously stayed hidden for three minutes. Both sample and cylinder now show estimates from the first two valid readings, using a time-weighted five-minute moving average (available history during startup).
Each side also has an independent “Measure long term leak” button and adjacent Reset button. Manual long term readings use a two-hour moving average, show the available window duration, and reset only the selected side. Start these measurements with a stable system; existing MEASURE/steady CONTROL eligibility is retained. Disconnecting clears manual measurements. Green confirmation remains based on uninterrupted monitoring time.
Updated English/Italian labels, documentation and changelog. The frozen legacy controller is unchanged.
Validation: 34 cross-platform tests and 7 repository checks passed; visually checked the real offscreen GUI with both long term measurements active. Includes tests for early UI output, rolling-window interpolation, irregular sample timing, independent resets, two-hour expiry and connection/mode resets.