Skip to content

Use png image instead of a vector image in AnimatedVisibility benchmark #5275

Closed
eymar wants to merge 1 commit intomasterfrom
ok/benchmark_animated_visibility_use_png
Closed

Use png image instead of a vector image in AnimatedVisibility benchmark #5275
eymar wants to merge 1 commit intomasterfrom
ok/benchmark_animated_visibility_use_png

Conversation

@eymar
Copy link
Member

@eymar eymar commented Apr 9, 2025

It's necessary in preparation for D8 benchmarks, since D8 environment misses some APIs including a parser of xml.

Also, make sure the web targets process the asynchronous fetch resources operations by yielding the event loop. Otherwise, a benchmark would skip a part of the workload.

Testing

N/A

Release Notes

N/A

…rk in preparation for D8 benchmarks

D8 environment misses some APIs including a parser of xml.

Also, make sure the web targets process the asynchronous fetch resources operations by yielding the event loop. Otherwise, a benchmark would skip a part of the workload.
@eymar eymar requested a review from pjBooms April 9, 2025 12:54
@eymar eymar changed the title Use png image instead of a vector image in AnimatedVisibility benchmark in preparation for D8 benchmarks Use png image instead of a vector image in AnimatedVisibility benchmark Apr 9, 2025
@eymar
Copy link
Member Author

eymar commented Apr 11, 2025

I'm closing this PR in favor of #5277 which contains the changes from this PR too.

@eymar eymar closed this Apr 11, 2025
eymar added a commit that referenced this pull request Apr 29, 2025
- Updated the README.md with the description of benchmarks (might be
useful to external parties)
- customized a runner for D8
- added a `warmupCount` parameter to `runBenchmarks`, because it's
required for Jetstream3 (a set of benchmarks) that there is no warmup
(they skip it)
- Added a smooth scroll variant of LazyGrid + variants with
LaunchedEffect in grid cells for coroutines usage.
- Added a new class `Config`. Object `Args` is simplified, it's purpose
is to simply parse the arguments and produce a instance of `Config`. The
usages of `Args` were replaced by `Config`, except the parsing in fun
main.
- Renamed Example1 to MultipleComponents
- Also added MultipleComponents-NoVectorGraphics for D8 - it's similar
to MultipleComponents but doesn't use vector icons.

Fixes https://youtrack.jetbrains.com/issue/CMP-6942

Note: it's based on this PR
#5275

## Testing
Manually run the benchmarks

## Release Notes
N/A
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.

1 participant