Skip to content

fix(parser): map LTF IterationsToFirstSuccess and DriftScore - #25

Open
rune-oai wants to merge 4 commits into
loop-eng:mainfrom
rune-oai:fix/11-map-ltf-convergence-fields
Open

rune-oai wants to merge 4 commits into
loop-eng:mainfrom
rune-oai:fix/11-map-ltf-convergence-fields

Conversation

@rune-oai

Copy link
Copy Markdown

Summary

LTF loop_summary.convergence already parsed iterations_to_first_success and drift_score, but only verification_pass_rate was copied into trace.Summary.

  • Add IterationsToFirstSuccess and DriftScore on Summary
  • Map them from the LTF summary event
  • Include them in JSON export (omitempty)

Fixes #11

Test plan

  • go test ./internal/parser/ -run MapsConvergenceFields -count=1

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.

LTF summary IterationsToFirstSuccess and DriftScore parsed but unused

1 participant