Skip to content

fix(prover): store simulation duration as ms#20460

Merged
spalladino merged 1 commit intomerge-train/spartanfrom
palla/fix-simulation-metric
Feb 12, 2026
Merged

fix(prover): store simulation duration as ms#20460
spalladino merged 1 commit intomerge-train/spartanfrom
palla/fix-simulation-metric

Conversation

@spalladino
Copy link
Contributor

Metric CIRCUIT_SIMULATION_DURATION was defined as int ms, but was stored as floating-point seconds. This caused errors INT value type cannot accept a floating-point value for aztec.circuit.simulation.duration, ignoring the fractional digits. in the logs.

This fixes it so simulation duration is stored in truncated ms. Alternative fix is to convert the metric type to floating-point seconds, as the other durations.

Metric `CIRCUIT_SIMULATION_DURATION` was defined as int ms, but was
stored as floating-point seconds. This caused errors `INT value type cannot accept a floating-point value for aztec.circuit.simulation.duration, ignoring the fractional digits.` in the logs.

This fixes it so simulation duration is stored in truncated ms.
Alternative fix is to convert the metric type to floating-point seconds,
as the other durations.
@spalladino spalladino merged commit b6641a0 into merge-train/spartan Feb 12, 2026
9 checks passed
@spalladino spalladino deleted the palla/fix-simulation-metric branch February 12, 2026 22:01
github-merge-queue bot pushed a commit that referenced this pull request Feb 13, 2026
BEGIN_COMMIT_OVERRIDE
chore(spartan): bots sync to the proposed chain in mbps net (#20432)
fix(grind-test): pass NAME_POSTFIX (#20437)
chore(e2e): replace link to slack conversation with summary (#20438)
feat: add cross-chain message bot (#20408)
chore(bot): fix tsconfig (#20464)
fix(prover): store simulation duration as ms (#20460)
fix(validator): remove hardcoded 10s timeout for previous block syncing
(#20462)
END_COMMIT_OVERRIDE
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