Skip to content

GSYE-900: Add Decimal to pay_as_bid_matching algorithm in order to co…#612

Merged
hannesdiedrich merged 2 commits intomasterfrom
bug/GSYE-900
Mar 12, 2026
Merged

GSYE-900: Add Decimal to pay_as_bid_matching algorithm in order to co…#612
hannesdiedrich merged 2 commits intomasterfrom
bug/GSYE-900

Conversation

@hannesdiedrich
Copy link
Member

…rrectly compare selected_energy to FLOATING_POINT_TOLERANCE

Reason for the proposed changes

Please describe what we want to achieve and why.

Proposed changes

GSY_WEB_BRANCH=master

…rrectly compare selected_energy to FLOATING_POINT_TOLERANCE
@codecov
Copy link

codecov bot commented Jan 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 67.61%. Comparing base (574835b) to head (545ef3e).
⚠️ Report is 4 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #612      +/-   ##
==========================================
+ Coverage   67.60%   67.61%   +0.01%     
==========================================
  Files          88       88              
  Lines        5569     5571       +2     
  Branches     1262     1262              
==========================================
+ Hits         3765     3767       +2     
  Misses       1622     1622              
  Partials      182      182              
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Member

@spyrostz spyrostz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@hannesdiedrich hannesdiedrich merged commit c385a7e into master Mar 12, 2026
4 checks passed
@hannesdiedrich hannesdiedrich deleted the bug/GSYE-900 branch March 12, 2026 13:57
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.

2 participants