esm icon indicating copy to clipboard operation
esm copied to clipboard

A minor issue in the example code for Forge Batch Executor Section

Open asceking opened this issue 9 months ago • 0 comments

Hi, thank you for the great work and the updates!

I directly ran the codes in Forge Batch Executor Section, and the exception would be thrown: ("embed_sequence() got an unexpected keyword argument 'model'",). This could be easily fixed by using consistent parameter names in embed_sequence and the call to execute_batch. (The example code uses client in embed_sequence but passes model in execute_batch.) Also it seems that LogitsOutput and ESMProteinError should be explicitly imported.

It's easy to debug, but still might cause confusion when trying to follow the example directly.

Just a tiny thing. Thanks again for your efforts on this project.

asceking avatar Apr 05 '25 19:04 asceking