Skip to content

Conversation

@kcz358
Copy link
Collaborator

@kcz358 kcz358 commented Sep 24, 2024

Before you open a pull-request, please check if a similar issue already exists or has been closed before.

When you open a pull-request, please be sure to include the following

  • A descriptive title: [xxx] XXXX
  • A detailed description

Thank you for your contributions!


We forgot to pass in the model name in the task manager, this could cause the specific kwargs to be useless. This PR is to fix this error

Also, it seems like we are always retrying in the claude. No wonder the speed is so slow.

@kcz358 kcz358 changed the title [Fix] Model name None in Task manager, mix eval model specific kwargs [Fix] Model name None in Task manager, mix eval model specific kwargs, claude retrying fix Sep 25, 2024
Copy link
Contributor

@Luodian Luodian left a comment

Choose a reason for hiding this comment

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

Good fix, thanks!

@Luodian Luodian merged commit dfbddf8 into main Sep 25, 2024
@Luodian Luodian deleted the fix/model_name_mix_evals branch September 25, 2024 07:52
KairuiHu pushed a commit that referenced this pull request Oct 24, 2024
…, claude retrying fix (#278)

* Fix task manager model name None issue

* Change model specific args in mix eval to lmms eval kwargs

* json dump indent 4

* lint

* Fix claude always retrying 5 time error
MichalCiesiolka pushed a commit to MichalCiesiolka/lmms-eval-llmzszl that referenced this pull request Apr 3, 2025
…, claude retrying fix (EvolvingLMMs-Lab#278)

* Fix task manager model name None issue

* Change model specific args in mix eval to lmms eval kwargs

* json dump indent 4

* lint

* Fix claude always retrying 5 time error
dadwadw233 pushed a commit to dadwadw233/lmms-eval that referenced this pull request Apr 28, 2025
…, claude retrying fix (EvolvingLMMs-Lab#278)

* Fix task manager model name None issue

* Change model specific args in mix eval to lmms eval kwargs

* json dump indent 4

* lint

* Fix claude always retrying 5 time error
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.

3 participants