search

Garden-AI / garden

https://garden-ai.readthedocs.io
MIT License
19 stars 4 forks source link

Bake DLHub package installs into DLHub containers #408

Closed WillEngler closed 7 months ago

WillEngler commented 7 months ago

On the existing demo endpoint, we currently run the following command after starting up a container.

pip install funcx==1.0.13 funcx-endpoint==1.0.13; pip install home_run==0.5.0; export PYTHONPATH="$PYTHONPATH:/app:/home/ubuntu"; mkdir -p /home/ubuntu/

This is needed for DLHub containers. To my knowledge, DLHub did not bake home_run or similar into its containers at publish time. So this is necessary for DLHub models to work.

This slows down Garden container start-up time. It also means that DLHub functions that work on the demo endpoint won't work on other K8s endpoints. I think we should run this command as an extra layer on the DLHub containers and host this "batteries included" version on the Garden Docker Hub account.

Related to #407.

Assumptions:

1. 2.

Acceptance Criteria

Given..., when..., then...