Skip to content

Enforces epsilon minimum bound checks - #3224

Open
Opt-Mucca wants to merge 1 commit into
latestfrom
epsilon-and-machine-underflow
Open

Enforces epsilon minimum bound checks#3224
Opt-Mucca wants to merge 1 commit into
latestfrom
epsilon-and-machine-underflow

Conversation

@Opt-Mucca

Copy link
Copy Markdown
Collaborator

Description

This re-implements #3203 and #3204 using std::numeric_limits<double>::epsilon()

Checklist

  • I have read the contributing guidelines
  • This PR targets the latest branch
  • Tests are passing
  • Documentation was updated where relevant
  • This PR is not primarily AI-generated (per the AI contributions policy in CONTRIBUTING.md)

@Opt-Mucca
Opt-Mucca requested a review from fwesselm August 20, 2026 16:57
@Opt-Mucca Opt-Mucca changed the title Add fixes Enforced epsilon minimum bound checks Aug 20, 2026
@Opt-Mucca Opt-Mucca changed the title Enforced epsilon minimum bound checks Enforces epsilon minimum bound checks Aug 20, 2026
@codecov

codecov Bot commented Aug 20, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.05%. Comparing base (2ed99be) to head (037d05d).
⚠️ Report is 23 commits behind head on latest.

Additional details and impacted files
@@            Coverage Diff             @@
##           latest    #3224      +/-   ##
==========================================
- Coverage   73.06%   73.05%   -0.01%     
==========================================
  Files         445      445              
  Lines      107661   107749      +88     
  Branches    17257    17264       +7     
==========================================
+ Hits        78659    78714      +55     
- Misses      28726    28759      +33     
  Partials      276      276              

☔ View full report in Codecov by Harness.
📢 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.

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.

1 participant