Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Updated some impeller benchmark urls#56721

Merged
gaaclarke merged 1 commit into
flutter-team-archive:mainfrom
gaaclarke:update-benchmarkmd
Nov 21, 2024
Merged

Updated some impeller benchmark urls#56721
gaaclarke merged 1 commit into
flutter-team-archive:mainfrom
gaaclarke:update-benchmarkmd

Conversation

@gaaclarke

Copy link
Copy Markdown
Contributor

The urls that use keys seem to atrophy quickly

Pre-launch Checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I read the Tree Hygiene wiki page, which explains my responsibilities.
  • I read and followed the Flutter Style Guide and the C++, Objective-C, Java style guides.
  • I listed at least one issue that this PR fixes in the description above.
  • I added new tests to check the change I am making or feature I am adding, or the PR is test-exempt. See testing the engine for instructions on writing and running engine tests.
  • I updated/added relevant documentation (doc comments with ///).
  • I signed the CLA.
  • All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@gaaclarke gaaclarke changed the title Updated some benchmark urls Updated some impeller benchmark urls Nov 19, 2024

@jonahwilliams jonahwilliams left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@gaaclarke gaaclarke added the autosubmit Merge PR when tree becomes green via auto submit App label Nov 20, 2024
@fanchou

fanchou commented Nov 20, 2024

Copy link
Copy Markdown

There may be a problem here, or I misunderstood? The same applies to new_gallery_opengles_impactr_transition_perf
WechatIMG6516
WechatIMG6517

@auto-submit auto-submit Bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Nov 20, 2024
@auto-submit

auto-submit Bot commented Nov 20, 2024

Copy link
Copy Markdown
Contributor

auto label is removed for flutter/engine/56721, due to - The status or check suite Mac mac_ios_engine has failed. Please fix the issues identified (or deflake) before re-applying this label.

@gaaclarke

Copy link
Copy Markdown
Contributor Author

There may be a problem here, or I misunderstood? The same applies to new_gallery_opengles_impactr_transition_perf

That's intentional for comparing opengls to vulkan

@gaaclarke
gaaclarke merged commit c2b2d1c into flutter-team-archive:main Nov 21, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 21, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 21, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 22, 2024
github-merge-queue Bot pushed a commit to flutter/flutter that referenced this pull request Nov 22, 2024
flutter-team-archive/engine@d1a0806...6f941c9

2024-11-21 98614782+auto-submit[bot]@users.noreply.github.com Reverts
"[Impeller] support GLES 3.0 MSAA without extension. (#56705)"
(flutter-team-archive/engine#56741)
2024-11-21 skia-flutter-autoroll@skia.org Roll Dart SDK from
dde57dc75c15 to b36e4d731d67 (1 revision) (flutter-team-archive/engine#56723)
2024-11-21 dacoharkes@google.com [native assets] Consume
`NativeAssetsManifest.json` (flutter-team-archive/engine#56727)
2024-11-21 jonahwilliams@google.com [Impeller] support GLES 3.0 MSAA
without extension. (flutter-team-archive/engine#56705)
2024-11-21 30870216+gaaclarke@users.noreply.github.com Updated some
impeller benchmark urls (flutter-team-archive/engine#56721)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC aaclarke@google.com,zra@google.com on the revert to ensure
that a human
is aware of the problem.

To file a bug in Flutter:
https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
nick9822 pushed a commit to nick9822/flutter that referenced this pull request Dec 18, 2024
The urls that use keys seem to atrophy quickly

## Pre-launch Checklist

- [x] I read the [Contributor Guide] and followed the process outlined
there for submitting PRs.
- [x] I read the [Tree Hygiene] wiki page, which explains my
responsibilities.
- [x] I read and followed the [Flutter Style Guide] and the [C++,
Objective-C, Java style guides].
- [x] I listed at least one issue that this PR fixes in the description
above.
- [x] I added new tests to check the change I am making or feature I am
adding, or the PR is [test-exempt]. See [testing the engine] for
instructions on writing and running engine tests.
- [x] I updated/added relevant documentation (doc comments with `///`).
- [x] I signed the [CLA].
- [x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel
on [Discord].

<!-- Links -->
[Contributor Guide]:
https://github.com/flutter/flutter/wiki/Tree-hygiene#overview
[Tree Hygiene]: https://github.com/flutter/flutter/wiki/Tree-hygiene
[test-exempt]:
https://github.com/flutter/flutter/wiki/Tree-hygiene#tests
[Flutter Style Guide]:
https://github.com/flutter/flutter/wiki/Style-guide-for-Flutter-repo
[C++, Objective-C, Java style guides]:
https://github.com/flutter/engine/blob/main/CONTRIBUTING.md#style
[testing the engine]:
https://github.com/flutter/flutter/wiki/Testing-the-engine
[CLA]: https://cla.developers.google.com/
[flutter/tests]: https://github.com/flutter/tests
[breaking change policy]:
https://github.com/flutter/flutter/wiki/Tree-hygiene#handling-breaking-changes
[Discord]: https://github.com/flutter/flutter/wiki/Chat
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Development

Successfully merging this pull request may close these issues.

4 participants