Skip to content

Conversation

@ydshieh
Copy link
Collaborator

@ydshieh ydshieh commented Feb 17, 2025

What does this PR do?

Time to say goodbye ~~~~~~~~

For tests/models/depth_pro/test_image_processing_depth_pro.py::DepthProImageProcessingTest::test_fast_is_faster_than_slow

(running 1000 times)

  • main: 3.7% failure
  • pr: 0%

For tests/models/rt_detr/test_image_processing_rt_detr.py::RtDetrImageProcessingTest::test_fast_is_faster_than_slow

(running 1000 times)

  • main: 2.4% failure
  • pr: 0.6 % failure (so we use is_flaky)

(haven't run for all processors yet, but from above check, this PR does its job in genral)

@HuggingFaceDocBuilderDev

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

Copy link
Contributor

@gante gante left a comment

Choose a reason for hiding this comment

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

LGTM, thank you for de-flaking it 💛

@ydshieh ydshieh merged commit 626666c into main Feb 17, 2025
26 checks passed
@ydshieh ydshieh deleted the fix_flaky_test_fast_is_faster_than_slow branch February 17, 2025 17:30
@yonigozlan
Copy link
Member

Thanks a lot for this!

zucchini-nlp pushed a commit to zucchini-nlp/transformers that referenced this pull request Feb 21, 2025
* fix

* fix

* fix

---------

Co-authored-by: ydshieh <[email protected]>
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.

6 participants