Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@courtneypacheco
Copy link
Contributor

Checklist:

  • Commit Message Formatting: Commit titles and messages follow guidelines in the
    conventional commits.
  • Changelog updated with breaking and/or notable changes for the next minor release.
  • Documentation has been updated, if necessary.
  • Unit tests have been added, if necessary.
  • Functional tests have been added, if necessary.
  • E2E Workflow tests have been added, if necessary.

In certain situations, linux_train.py fails with: "UserWarning: return_dict_in_generate is NOT set to True, but output_logits is. When return_dict_in_generate is not True, output_logits is ignored."

To fix this failure, I have added return_dict_in_generate=True as an input to model.generate().

@mergify mergify bot added the ci-failure PR has at least one CI failure label Apr 11, 2025
@courtneypacheco courtneypacheco force-pushed the fix-logits-issue-in-linux-train branch from 4e08dad to 2ca3b10 Compare April 11, 2025 02:12
@mergify mergify bot added ci-failure PR has at least one CI failure and removed ci-failure PR has at least one CI failure labels Apr 11, 2025
@courtneypacheco courtneypacheco force-pushed the fix-logits-issue-in-linux-train branch from 2ca3b10 to 8af5e34 Compare April 11, 2025 02:15
@mergify mergify bot added ci-failure PR has at least one CI failure and removed ci-failure PR has at least one CI failure labels Apr 11, 2025
@courtneypacheco courtneypacheco force-pushed the fix-logits-issue-in-linux-train branch from 8af5e34 to c5c4d75 Compare April 11, 2025 03:43
@mergify mergify bot removed the ci-failure PR has at least one CI failure label Apr 11, 2025
@courtneypacheco courtneypacheco force-pushed the fix-logits-issue-in-linux-train branch from c5c4d75 to 64ddc93 Compare April 11, 2025 03:46
@mergify mergify bot added the ci-failure PR has at least one CI failure label Apr 11, 2025
@courtneypacheco courtneypacheco force-pushed the fix-logits-issue-in-linux-train branch from 64ddc93 to 57bac27 Compare April 11, 2025 04:38
@mergify mergify bot removed the ci-failure PR has at least one CI failure label Apr 11, 2025
@courtneypacheco courtneypacheco force-pushed the fix-logits-issue-in-linux-train branch from 57bac27 to f8982d6 Compare April 11, 2025 04:52
@mergify mergify bot added the ci-failure PR has at least one CI failure label Apr 11, 2025
In certain situations, `linux_train.py` fails with: "UserWarning: `return_dict_in_generate` is NOT set to `True`, but `output_logits` is. When `return_dict_in_generate` is not `True`, `output_logits` is ignored."

To fix this failure, I have added `return_dict_in_generate=True` as an input to `model.generate()`.

Signed-off-by: Courtney Pacheco <[email protected]>
@courtneypacheco courtneypacheco force-pushed the fix-logits-issue-in-linux-train branch from f8982d6 to b0cd529 Compare April 11, 2025 05:26
@RobotSail
Copy link
Member

@courtneypacheco We have a PR to fix this in #3264

@booxter
Copy link
Contributor

booxter commented Apr 16, 2025

Please link to #3289 if you intend to continue on this PR to fix the issue in linux-train flow.

@booxter
Copy link
Contributor

booxter commented Apr 28, 2025

CI issue should be addressed with #3307 I believe. If we still have issues with the job, please reopen and refresh the patch. For now, I'm closing it.

@booxter booxter closed this Apr 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-failure PR has at least one CI failure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants