Great work! Left a small feedback

#1
by merve HF staff - opened
scikit-learn org

Great work @LenixC ! I feel like it would be even better if you'd put the hyperparameter sliders side by side to reduce the space, and since they're sliders it would be better to remove the button and replace it with event listeners on sliders (e.g. instead of button.click(function, inputs, outputs), you'd do slider.change(function, other inputs, outputs)). You can see an example below:

import gradio as gr

def welcome(name):
    return f"Welcome to Gradio, {name}!"

with gr.Blocks() as demo:
    gr.Markdown(
    """
    # Hello World!
    Start typing below to see the output.
    """)
    inp = gr.Textbox(placeholder="What is your name?")
    out = gr.Textbox()
    inp.change(welcome, inp, out)

demo.launch()
scikit-learn org

Thanks, @merve ! Sliders have been moved and have been given listeners.

Sign up or log in to comment