-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Lite load perf ci #8222
Lite load perf ci #8222
Conversation
🪼 branch checks and previews
Install Gradio from this PR pip install https://gradio-builds.s3.amazonaws.com/ed2f867ac43dd051d5f1712b39ea24a3105e1542/gradio-4.37.2-py3-none-any.whl Install Gradio Python Client from this PR pip install "gradio-client @ git+https://github.com/gradio-app/gradio@ed2f867ac43dd051d5f1712b39ea24a3105e1542#subdirectory=client/python" Install Gradio JS Client from this PR npm install https://gradio-builds.s3.amazonaws.com/ed2f867ac43dd051d5f1712b39ea24a3105e1542/gradio-client-1.2.1.tgz |
🦄 change detectedThis Pull Request includes changes to the following packages.
With the following changelog entry.
Maintainers or the PR author can modify the PR title to modify this entry.
|
da6bf1d
to
df4c29f
Compare
d9ee9ce
to
b246886
Compare
@pngwn @freddyaboulton What do you think is an appropriate way to inform the performance measurement result to us from the CI runs? |
Depends on how we want to display it / how much info there is to display. We could add an additional entry to the table, or we could add another chunk of markdown to the comment, after or before the How do you think it would be best to display the info? |
@pngwn I think showing a new Markdown section would be good. |
In that case, it might not need any changes to the action. It the section has a new |
@pngwn I tried it (passing |
In this case we actually don't want to add an entry to the table I think, which is mostly what that switch handles iirc. I will refamiliarise myself with the action and respond shortly. |
What's the status of this @whitphx? |
@pngwn Do you have any idea about how to post messages to the PR pages from the test-functional workflow? |
I forgot to respond shortly. |
I think it's good to merge now. We can follow up the PR comment in a new PR. |
Let's go ahead and merge in for the release! Thanks @whitphx |
Description
Closes: #8193