Skip to content

perf: ICE - solver input linear queue - #1554

Merged
enthusiastmartin merged 1 commit into
masterfrom
fix/ice-solver-input
Oct 17, 2026
Merged

enthusiastmartin merged 1 commit into
masterfrom
fix/ice-solver-input

Conversation

@enthusiastmartin

Copy link
Copy Markdown
Member

apply_conditional_offers rescanned the whole intent list for every conditional intent in every
round, so solver_intents() grew quadratically with the unbounded global queue. 10k intents took
3.16 s in release, more than the 2 s block time, so the ICE solver would miss every slot.

The pass now builds per-pair flow totals once and updates them as intents are withheld or capped.
Same fixpoint, ordering and arithmetic, so the output is identical. 10k intents: 52.2 s → 0.145 s (debug).

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Claude Code Review

Claude Code Review is paused for this repository because its GitHub organization is not linked to your Claude organization.

To turn it back on, one person needs both roles: owner of the GitHub organization (or the account owner, for a personal repository) and admin or owner of your Claude organization. If that is not you, pass this on to someone who is.

Open Organization settings, Git providers and click Link next to the GitHub organization's name. If it is not listed, click Enable GitHub integration or Connect your GitHub first, whichever the page shows.

New pull requests are then reviewed as before. To get a review on one that is already open, someone with write access can comment @claude review.

Tip: disable this comment in your organization's Code Review settings.

@github-actions

Copy link
Copy Markdown

Crate versions that have been updated:

  • pallet-intent: v1.13.0 -> v1.13.1
  • hydradx-runtime: v454.0.0 -> v455.0.0

Runtime version has been increased.

@enthusiastmartin
enthusiastmartin merged commit 6d7da8c into master Oct 17, 2026
10 of 11 checks passed
@github-actions

Copy link
Copy Markdown

Quick benchmark at commit fc2c4f2 has been executed successfully.
View results

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