gronod/ICCery
1
0
Fork 0
Code Issues 2 Pull Requests Actions Packages Projects Releases 6 Wiki Activity

feat(stage3): Multi-pass target measurement averaging with Argyll average (fixes #87) #99

Merged
gronod merged 4 commits from feat/87-measurement-averaging into development 2026-08-25 17:43:21 +01:00
Conversation 0 Commits 4 Files Changed 4
+152 -5
gronod commented 2026-08-25 17:30:14 +01:00
Owner
Copy Link
Copy Source

Summary of Changes

  • Backend (src-tauri/src/commands.rs, src-tauri/src/lib.rs):
    • Implemented AverageConfig and build_average_args to invoke Argyll average binary with -v, input .ti3 file paths, and output destination .ti3.
    • Registered commands::run_average in Tauri invoke handler.
    • Added unit test test_build_average_args to test suite.
  • UI (src/index.html):
    • Added #chartreadAveragingPanel to Stage 3 (Measure Target) displaying pass history list, "➕ Measure Another Sheet" button, and "✓ Average Passes & Proceed" button.
  • Frontend Logic (src/js/chartread.js):
    • Implemented multi-pass sheet measurement session management.
    • Handled automated averaging calculation via run_average before transitioning into Stage 4.

Fixes #87.

### Summary of Changes - **Backend (`src-tauri/src/commands.rs`, `src-tauri/src/lib.rs`)**: - Implemented `AverageConfig` and `build_average_args` to invoke Argyll `average` binary with `-v`, input `.ti3` file paths, and output destination `.ti3`. - Registered `commands::run_average` in Tauri invoke handler. - Added unit test `test_build_average_args` to test suite. - **UI (`src/index.html`)**: - Added `#chartreadAveragingPanel` to Stage 3 (`Measure Target`) displaying pass history list, "➕ Measure Another Sheet" button, and "✓ Average Passes & Proceed" button. - **Frontend Logic (`src/js/chartread.js`)**: - Implemented multi-pass sheet measurement session management. - Handled automated averaging calculation via `run_average` before transitioning into Stage 4. Fixes #87.
gronod added the Kind/Feature
Priority
High
2
 labels 2026-08-25 17:30:14 +01:00
gronod added 4 commits 2026-08-25 17:30:16 +01:00
feat(backend): add run_average command and AverageConfig (fixes #87) 454ee3488d
feat(backend): register run_average command in lib.rs (fixes #87) b215c72162
feat(ui): add multi-pass averaging controls to Stage 3 HTML (fixes #87) a5840baa76
feat(stage3): multi-pass sheet measurement and averaging workflow (fixes #87)
Build Linux Packages / Build Linux (pull_request) Successful in 9m5s
Details
Build Windows Packages / Build Windows (pull_request) Successful in 7m18s
Details
8a74501aff
gronod merged commit 53be326475 into development 2026-08-25 17:43:21 +01:00
gronod deleted branch feat/87-measurement-averaging 2026-08-25 17:43:22 +01:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
Compat/Breaking
Breaking change that won't be backward compatible
 Kind/Bug
Something is not working
 Kind/Documentation
Documentation changes
 Kind/Enhancement
Improve existing functionality
 Kind/Feature
New functionality
 Kind/Security
This is security issue
 Kind/Testing
Issue or pull request related to testing
Priority
Critical
1
The priority is critical
Priority
High
2
The priority is high
Priority
Low
4
The priority is low
Priority
Medium
3
The priority is medium
Reviewed
Confirmed
1
Issue has been confirmed
Reviewed
Duplicate
2
This issue or pull request already exists
Reviewed
Invalid
3
Invalid issue
Reviewed
Won't Fix
3
This issue won't be fixed
Status
Abandoned
3
Somebody has started to work on this but abandoned work
Status
Blocked
1
Something is blocking this issue or pull request
Status
Need More Info
2
Feedback is required to reproduce issue or to continue work
No labels Kind/Feature
Priority
High
2
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
gronod
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: gronod/ICCery#99