Skip to content

Conversation

@mrfaizal
Copy link
Contributor

@mrfaizal mrfaizal commented Nov 14, 2025

Fix flaky tests due to imprecision in floating point calculation and performance test setup

@product-auto-label product-auto-label bot added the size: xs Pull request size is extra small. label Nov 14, 2025
Avoid floating point conversion imprecision
Fix linter error with black
Ensure URL encoding takes enough time so that cached version is measurably 10x faster
@mrfaizal mrfaizal changed the title WIP: Fix floating point test error fix: flaky tests due to imprecision in floating point calculation and performance test setup Nov 14, 2025
@mrfaizal mrfaizal marked this pull request as ready for review November 14, 2025 05:39
@mrfaizal mrfaizal requested review from a team as code owners November 14, 2025 05:39
@mrfaizal mrfaizal assigned vchudnov-g and unassigned Linchin Nov 14, 2025
coryan
coryan previously approved these changes Nov 14, 2025
Change code to skip flaky test and add a new test that uses a constant timeout instead of a number to check that override works
@product-auto-label product-auto-label bot added size: s Pull request size is small. and removed size: xs Pull request size is extra small. labels Nov 15, 2025
@mrfaizal mrfaizal requested a review from vchudnov-g November 15, 2025 01:00
Copy link
Contributor

@vchudnov-g vchudnov-g left a comment

Choose a reason for hiding this comment

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

Please do update the PR description to be more informative

@mrfaizal mrfaizal merged commit 9340408 into main Nov 17, 2025
48 checks passed
@mrfaizal mrfaizal deleted the fix-test branch November 17, 2025 19:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: s Pull request size is small.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants