Skip to content

Enhanced Gradio UI#89

Open
ParisNeo wants to merge 15 commits into
genmoai:mainfrom
ParisNeo:main
Open

Enhanced Gradio UI#89
ParisNeo wants to merge 15 commits into
genmoai:mainfrom
ParisNeo:main

Conversation

@ParisNeo

Copy link
Copy Markdown

Enhanced Gradio interface UI
Added the possibility to select the host and port number when launching the gradio service
Added the possibility to select a specific GPU when launching the gradio service

Comment thread demos/gradio_ui.py Outdated
demo.launch()
@click.option("--lora_path", required=False, help="Path to the lora file.")
@click.option("--cpu_offload", is_flag=True, help="Whether to offload model to CPU.")
@click.option("--gpu", default=None, help="Specify the GPU to use (e.g., 'cuda:0').")

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It seems like the gpu option is used to set CUDA_VISIBLE_DEVICES, so the help text is incorrect (cuda:0 isn't valid). Could you remove the option? Users can set CUDA_VISIBLE_DEVICES manually.

@ParisNeo

ParisNeo commented Dec 1, 2024

Copy link
Copy Markdown
Author

You are right. I'll make the change. Thanks

@ParisNeo
ParisNeo requested a review from paras-genmo December 2, 2024 07:51
@ParisNeo

Copy link
Copy Markdown
Author

Hi, can you please review my pull request?
Best regards

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