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

Live portrait runner #199

Open
wants to merge 29 commits into
base: main
Choose a base branch
from

Conversation

JJassonn69
Copy link
Contributor

This pull request includes the new method of making container for different pipelines by first building the base image and building off the the live-portrait container.

docker build -t livepeer/ai-runner:base .

docker build -f docker/Dockerfile.live-portrait -t livepeer/ai-runner:live-portrait .

While building the specific docker image for live.portrait, it will automatically pull wheel from https://github.com/JJassonn69/liveportrait and install all the necessary dependencies.

@JJassonn69 JJassonn69 marked this pull request as draft September 20, 2024 18:28
@JJassonn69 JJassonn69 marked this pull request as ready for review September 20, 2024 19:31
Copy link
Collaborator

@pschroedl pschroedl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If we resolve the merge conflicts here by rebasing or merging in changes from main, then I think we're good to merge. Thanks @JJassonn69 !

runner/app/routes/live_portrait.py Outdated Show resolved Hide resolved
runner/Dockerfile Outdated Show resolved Hide resolved
Copy link
Collaborator

@pschroedl pschroedl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We will need to add to our github actions to configure our CI to build the pipeline-specific image, as done here for Sam2:

https://github.com/livepeer/ai-worker/blob/main/.github/workflows/ai-runner-pipelines-docker.yaml

@rickstaa Unless @JJassonn69 wants to take this on, would you like to plan on a followup PR to be merged before the corresponding go-livepeer PR?

Copy link
Collaborator

@pschroedl pschroedl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lets get main merged in to resolve conflicts and the bindings regenerated - otherwise LGTM.

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

Successfully merging this pull request may close these issues.

2 participants