Skip to content

[CircleCI] skip test_assisted_decoding_sample for everyone#27511

Merged
ArthurZucker merged 9 commits intomainfrom
skip-ci
Nov 15, 2023
Merged

[CircleCI] skip test_assisted_decoding_sample for everyone#27511
ArthurZucker merged 9 commits intomainfrom
skip-ci

Conversation

@ArthurZucker
Copy link
Collaborator

What does this PR do?

cc @gante I'm having a look at why this is failing but skipping for now. (standalone works well)

@HuggingFaceDocBuilderDev
Copy link

HuggingFaceDocBuilderDev commented Nov 15, 2023

The documentation is not available anymore as the PR was closed or merged.

@gante
Copy link
Contributor

gante commented Nov 15, 2023

@ArthurZucker this PR should fix it!

I decided to rely on the tests instead of asking the contributor to double-check, but forgot that the tests were stochastic -- they rely on sample. The PR that caused this crash had a lucky CI run, so the problem got merged while undetected.

In other words, I should have been more careful :)

EduardoPach pushed a commit to EduardoPach/transformers that referenced this pull request Nov 19, 2023
…face#27511)

* skip 4 tests

* nits

* style

* wow it's not my day

* skip new failing tests

* style

* skip for NLLB MoE as well

* skip `test_assisted_decoding_sample` for everyone
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.

4 participants