Skip to content
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

🐈 Task: Docker build for ersilia-pack #1154

Open
5 tasks done
DhanshreeA opened this issue Jun 6, 2024 · 7 comments
Open
5 tasks done

🐈 Task: Docker build for ersilia-pack #1154

DhanshreeA opened this issue Jun 6, 2024 · 7 comments
Assignees

Comments

@DhanshreeA
Copy link
Member

DhanshreeA commented Jun 6, 2024

Summary

Create and test Docker builds for ersilia-pack with a few models currently built with bentoml.

Objective(s)

Models to build and test:

  • eos80ch βœ…
  • eos4qda ❌
  • eos69e6 ❌
  • eos2r5a ❌
  • eos7w6n ❌

Documentation

No response

@DhanshreeA
Copy link
Member Author

eos4qda and eos69e6 result in an Internal Server Error when making a prediction request. eos2r5a and eos7w6n do not have a server running.

@DhanshreeA
Copy link
Member Author

For now, it's best to just focus on this step by step. I will try to inspect how ersilia-pack is fairing with respect to just the model eos2r5a and then proceed to inspecting Docker builds. We can ignore the rest of the failing models in this list just for now.

@GemmaTuron
Copy link
Member

Hi @DhanshreeA

Before we consider the Ersilia Pack complete, it would be good to test and build all the above mentioned models. Please update when done and close the issue if all is now working.

@DhanshreeA
Copy link
Member Author

This is presently achieved here.

And documented here.

Cautionary note: Ersilia Pack dockerization is only supported from Python versions 3.8-3.12. To make older models work with ersilia pack dockerization, it is often possible that we might need to refactor the Dockerfile to specify dependencies in the format that ersilia pack expects.

@GemmaTuron
Copy link
Member

Hi @DhanshreeA
Is this issue good to close?

@GemmaTuron
Copy link
Member

Hi @DhanshreeA

Please update this issue

1 similar comment
@GemmaTuron
Copy link
Member

Hi @DhanshreeA

Please update this issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In Progress
Development

No branches or pull requests

2 participants