Skip to content

Update iOS scenario app test goldens to match changes from https://github.com/flutter/flutter/pull/182662#186390

Open
jason-simmons wants to merge 1 commit into
flutter:masterfrom
jason-simmons:pr_182662_goldens
Open

Update iOS scenario app test goldens to match changes from https://github.com/flutter/flutter/pull/182662#186390
jason-simmons wants to merge 1 commit into
flutter:masterfrom
jason-simmons:pr_182662_goldens

Conversation

@jason-simmons
Copy link
Copy Markdown
Member

That PR affected golden image tests that use platform views. But the images had not been updated in the PR because errors in the "Mac mac_unopt" suite were suppressed on the dashboard when the PR landed.

…2662

That PR affected golden image tests that use platform views.  But the
images had not been updated in the PR because errors in the
"Mac mac_unopt" suite were suppressed on the dashboard when the PR
landed.
@jason-simmons jason-simmons requested a review from b-luk May 12, 2026 04:22
@flutter-dashboard flutter-dashboard Bot added the CICD Run CI/CD label May 12, 2026
@gemini-code-assist
Copy link
Copy Markdown
Contributor

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@github-actions github-actions Bot added the engine flutter/engine related. See also e: labels. label May 12, 2026
Copy link
Copy Markdown
Contributor

@b-luk b-luk left a comment

Choose a reason for hiding this comment

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

Thanks, Jason!

If even these changes are above the RMSE threshold, I wonder what it takes to be under the threshold.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CICD Run CI/CD engine flutter/engine related. See also e: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants