Missing model.eval() in main README

facebookresearch / esm

Evolutionary Scale Modeling (esm): Pretrained language models for proteins

MIT License

3.26k stars 643 forks source link

Missing model.eval() in main README #169

Closed samgelman closed 2 years ago

samgelman commented 2 years ago

First, thanks for making these models available and easily accessible! The "quick start" example in the main README is missing a call to model.eval().

samgelman commented 2 years ago

I just searched through the closed issues, and I see that was intentional. Might still be good to clarify on the README. Otherwise, feel free to close this issue! Thanks!

tomsercu commented 2 years ago

Thanks for flagging, yes it has lead to confusion and worth including even in the minimal quickstart example.