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

add gradio app #53

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

add gradio app #53

wants to merge 1 commit into from

Conversation

jadechoghari
Copy link

Hey @gcorso, amazing work! 🔥😍

This PR makes it easier for developers to build on the go with Gradio (just one line of code).
code by @duerrsimon with some tiny changes

@jsilter
Copy link
Collaborator

jsilter commented Dec 3, 2024

Hi @jadechoghari thanks for the PR!

I do have a few comments before merging it in:

  1. Could you add a bit more description to the README section, something like:

Launch a local web interface using gradio:

python gradio_app.py

Then open a browser and go to https://localhost:8000

  1. Any gradio-specific dependencies should be added as optional under pyproject.toml, eg:
    [project.optional-dependencies]
    lint = ["ruff"]
    test = ["pytest", "requests"]
    gradio = ["gradio", "gradio_molecule3d", "gradio_cofoldinginput"]

  2. The URLs should be imported from the boltz package, and the cache path should dynamically retrieve the users home directory.

  3. I'm thinking gradio_app.py should go in the "src" directory

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