Vahe1994 / SpQR

Apache License 2.0
515 stars 40 forks source link

Drop --no_cache arg in lmeval.py #28

Closed poedator closed 1 year ago

poedator commented 1 year ago

--no_cache was always needed. Without this option the code would fail due to incompatibility in lm-eval-harness. In lm_eval/evaluate simple_evaluate() accepts model or string with model name. However here in the if branch without no_cache the code adds model to string. Since always pass loaded model there - adding it to string causes error. This is an issue in the older lm-eval harness code that we copied here. So I hardcoded it to be always True.

In the latest LM-eval code this error is already fixed with my commit.