Skip to content

Comments

Standardise constraint residual/normalised residual calculations#4066

Merged
je-cook merged 2 commits intomainfrom
standardise-constraints
Feb 19, 2026
Merged

Standardise constraint residual/normalised residual calculations#4066
je-cook merged 2 commits intomainfrom
standardise-constraints

Conversation

@timothy-nunn
Copy link
Collaborator

@timothy-nunn timothy-nunn commented Jan 28, 2026

Standardises how we calculate normalised residuals and errors for constraints for our three types of constraints: geq, eq, leq

@timothy-nunn timothy-nunn force-pushed the standardise-constraints branch from b38e1f1 to aa68eee Compare January 28, 2026 17:30
@codecov-commenter
Copy link

codecov-commenter commented Jan 28, 2026

Codecov Report

❌ Patch coverage is 85.78199% with 30 lines in your changes missing coverage. Please review.
✅ Project coverage is 46.86%. Comparing base (2230706) to head (3034787).

Files with missing lines Patch % Lines
process/constraints.py 87.01% 27 Missing ⚠️
process/scan.py 0.00% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4066      +/-   ##
==========================================
+ Coverage   46.83%   46.86%   +0.02%     
==========================================
  Files         136      136              
  Lines       29297    29228      -69     
==========================================
- Hits        13722    13698      -24     
+ Misses      15575    15530      -45     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@timothy-nunn timothy-nunn force-pushed the standardise-constraints branch from aa68eee to 6fb416c Compare February 3, 2026 13:48
@timothy-nunn timothy-nunn marked this pull request as ready for review February 11, 2026 16:44
@timothy-nunn timothy-nunn requested a review from a team as a code owner February 11, 2026 16:44
@kj5248
Copy link
Collaborator

kj5248 commented Feb 18, 2026

Have tested and looked at, it seems reasonable. All values are now accurate and negative for all correct (passing) constraints

@timothy-nunn timothy-nunn assigned je-cook and unassigned je-cook Feb 18, 2026
@timothy-nunn timothy-nunn linked an issue Feb 18, 2026 that may be closed by this pull request
@timothy-nunn timothy-nunn force-pushed the standardise-constraints branch from 6fb416c to 3034787 Compare February 19, 2026 11:42
@timothy-nunn timothy-nunn requested a review from je-cook February 19, 2026 11:42
@je-cook je-cook merged commit 2812d00 into main Feb 19, 2026
8 of 10 checks passed
@je-cook je-cook deleted the standardise-constraints branch February 19, 2026 14:05
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

Successfully merging this pull request may close these issues.

Standardise constraint code

4 participants