Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add test that verifies that the generation config passed in at model.predict() is used correctly. #3523

Merged
merged 3 commits into from
Aug 11, 2023
Merged

Add test that verifies that the generation config passed in at model.predict() is used correctly. #3523

merged 3 commits into from
Aug 11, 2023

Conversation

justinxzhao
Copy link
Collaborator

No description provided.

@github-actions
Copy link

Unit Test Results

  6 files  ±0    6 suites  ±0   1h 4m 39s ⏱️ - 3m 44s
34 tests ±0  28 ✔️  - 1    5 💤 ±0  1 ❌ +1 
88 runs  ±0  71 ✔️  - 1  16 💤 ±0  1 ❌ +1 

For more details on these failures, see this check.

Results for commit 071865c. ± Comparison against base commit 3a8d65e.

@justinxzhao justinxzhao merged commit 095a1ad into master Aug 11, 2023
15 checks passed
@justinxzhao justinxzhao deleted the generation_config_test branch August 11, 2023 21:08
arnavgarg1 pushed a commit that referenced this pull request Aug 11, 2023
@justinxzhao @arnavgarg1
Add test that verifies that the generation config passed in at model.…
c545285 
…predict() is used correctly. (#3523)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@arnavgarg1 arnavgarg1 arnavgarg1 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@justinxzhao @arnavgarg1