Skip to content

Update git hash for flame-engine/flame - #493

Open
spydon wants to merge 1 commit into
flutter:mainfrom
spydon:flame-update-hash
Open

spydon wants to merge 1 commit into
flutter:mainfrom
spydon:flame-update-hash

Conversation

@spydon

@spydon spydon commented Sep 22, 2026 •

Copy link
Copy Markdown
Contributor

Updates the pinned commit for registry/flame.test from 74721f09 to c35f9d80, the current head of flame-engine/flame main.

This picks up two changes that make the Flame customer test deterministic:

Picks up flame-engine/flame#4051, which fixes the solveQuadratic random
test tolerance that made the customer test fail for some seeds, and
flame-engine/flame#4052, which pins the randomized tests to a fixed seed
in the customer testing run so it stays deterministic.

@jtmcdole jtmcdole left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

💵 The CodeFu 1024-Note: Collector's Edition

The LGTM exchange rates are currently trading at:

  • 1 LGTM = Golden images match closely enough if I squint.
  • 2 LGTMs = I checked that this won't break the devicelab benchmarks.
  • 3 LGTMs = I trust that this C++ pointer won't leak memory every frame.
  • 4 LGTMs = CI is green on the first try without a single retry or shard failure.

💵 Value officially backed by the full faith, credit, and compute budget of the Flutter EngProd Infrastructure.

codefu-cash2

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