Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: improve playwright test performance slightly by avoiding base64 conversion #2064

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

JoCa96
Copy link
Collaborator

@JoCa96 JoCa96 commented Nov 11, 2024

improve playwright test performance slightly by avoiding base64 conversion

Copy link

changeset-bot bot commented Nov 11, 2024

⚠️ No Changeset found

Latest commit: 0960aef

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@JoCa96 JoCa96 force-pushed the joca96/chore-screenshot-withoutbase64 branch from fb6275b to 1e3285e Compare November 11, 2024 18:13
@JoCa96 JoCa96 force-pushed the joca96/chore-screenshot-withoutbase64 branch from 6ca02e4 to d022a56 Compare November 13, 2024 11:19
@JoCa96 JoCa96 changed the title use int8array instead of base64 img chore: improve playwright test performance slightly by avoiding base64 conversion Nov 13, 2024
@JoCa96 JoCa96 marked this pull request as ready for review November 13, 2024 11:20
@JoCa96 JoCa96 requested a review from a team as a code owner November 13, 2024 11:20
Copy link
Collaborator

@larsrickert larsrickert left a comment

Choose a reason for hiding this comment

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

What's the performance improvement on this? 😅

@larsrickert larsrickert self-assigned this Nov 13, 2024
@JoCa96
Copy link
Collaborator Author

JoCa96 commented Nov 13, 2024

@larsrickert Well, no base64 encoding and decoding. The bytes are now sent fresh and raw :D

@JoCa96 JoCa96 enabled auto-merge (squash) November 13, 2024 17:36
@larsrickert
Copy link
Collaborator

@larsrickert Well, no base64 encoding and decoding. The bytes are now sent fresh and raw :D

Looking at the CI speeds, it looks like there are not performance improvements or even disadavantages :/
I am not sure if this improvement (if any) is worth the code complexity at this point

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