Skip to content

Conversation

@gante
Copy link
Contributor

@gante gante commented Nov 19, 2024

WIP -- now that generation tests are being run, we can notice a few broken ones 👀

Generation tests that were not part of the model mixin tests were not being fetched. This PR fixes it, by allowing each job to apply multiple separate filters to then run the union of all filtered tests.

This PR:

  • Enables non-mixin generation tests
  • Fixes failing generation tests

👉 New files being fetched:
test_preparation/tests_generate_test_list.txt
Screenshot 2024-11-19 at 14 35 41

👉 dummy modification in a generation test triggering a test run:
(early commit before tests were fixed)
Screenshot 2024-11-19 at 14 38 12

@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.

@@ -1,56 +0,0 @@
{% set name = "transformers" %}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this file is 4yr old and has very old requirements -- I'm assuming this was used in an older CI :D

out = model.generate(input_ids, output_logits=True, output_scores=True, return_dict_in_generate=True)
out_with_temp = model.generate(
input_ids,
temperature=0.5,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

(this change is to trigger the tests in this file)

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