Skip to content

[SEI-9824][SEI-9825] Update oracle MidBlock logic#2251

Merged
udpatil merged 3 commits intomainfrom
uday/oracle-tally
Jul 23, 2025
Merged

[SEI-9824][SEI-9825] Update oracle MidBlock logic#2251
udpatil merged 3 commits intomainfrom
uday/oracle-tally

Conversation

@udpatil
Copy link
Copy Markdown
Collaborator

@udpatil udpatil commented Jul 22, 2025

Describe your changes and provide context

This updates oracle tally logic to behave properly in edge case scenarios. Additionally, we rework the validator iteration to be more performant for cases with larger numbers of total validators with a limited active set.

Testing performed to validate your change

Added unit tests to verify behavior

@udpatil udpatil changed the title Update oracle tally logic [SEI-9824] Update oracle tally logic Jul 22, 2025
@udpatil udpatil force-pushed the uday/oracle-tally branch from 261cb57 to 074f0fc Compare July 22, 2025 22:44
@codecov
Copy link
Copy Markdown

codecov bot commented Jul 23, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 60.11%. Comparing base (8592896) to head (0255508).
Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2251      +/-   ##
==========================================
+ Coverage   60.10%   60.11%   +0.01%     
==========================================
  Files         309      309              
  Lines       29988    29995       +7     
==========================================
+ Hits        18024    18032       +8     
+ Misses      10690    10687       -3     
- Partials     1274     1276       +2     
Files with missing lines Coverage Δ
x/oracle/abci.go 100.00% <100.00%> (ø)
x/oracle/tally.go 91.42% <100.00%> (+1.59%) ⬆️

... and 3 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@udpatil udpatil changed the title [SEI-9824] Update oracle tally logic [SEI-9824][SEI-9825] Update oracle MidBlock logic Jul 23, 2025
@udpatil udpatil force-pushed the uday/oracle-tally branch from 5360fb2 to 05c7366 Compare July 23, 2025 17:32
@udpatil udpatil force-pushed the uday/oracle-tally branch from 05c7366 to 0255508 Compare July 23, 2025 19:30
@udpatil udpatil merged commit b16814c into main Jul 23, 2025
61 of 63 checks passed
@udpatil udpatil deleted the uday/oracle-tally branch July 23, 2025 21:36
yzang2019 added a commit that referenced this pull request Jul 24, 2025
* main:
  Optimization: CreateAccount only clears state if code hash exists (#2255)
  chore: bump btcec to v2.3.2, x/crypto to v0.31.0 (#2238)
  Use legacy transaction decoder for historical height (#2234)
  Make flushing receipt synchronous (#2250)
  [SEI-9824][SEI-9825] Update oracle MidBlock logic (#2251)
  Fix data race in price-feeder websocket controller (#2256)
  Add tests for price feeder providers (#2253)
  remove arm64 target from CI due to slow running (#2254)
  Harden solo precompile (#2236)
  Add CODEOWNERS (#2237)
  Require MsgClaim sender to match signer (#2232)
  Remove writeset estimation to alleviate AccAddress mutex contention (#2239)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants