-
Notifications
You must be signed in to change notification settings - Fork 7
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.
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
[TLM] Update max_tokens for TLM to match provider limits #324
Conversation
/test-tlm |
/rerun-test-tlm |
/rerun-failed-test-tlm |
/test-tlm |
we should also change the unit test fixture here cleanlab-studio/tests/tlm/conftest.py Line 90 in 90fb5cf
if 4096 is too large for claude models for now, we can keep that 512 if the model is claude and change to 4096 if the model is gpt |
Ensure final changes made to the TLM code are tested before merging. To run the TLM tests, comment |
/test-tlm |
Ensure final changes made to the TLM code are tested before merging. To run the TLM tests, comment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
changes LGTM. let's verify tests can pass after the backend changes deployed before merge.
/test-tlm |
/test-tlm |
No description provided.