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.

Sign up for GitHub

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

Jump to bottom

Fix to work with updated gradio #30

Merged
merged 3 commits into from
Jan 11, 2024
Merged

Fix to work with updated gradio #30

merged 3 commits into from
Jan 11, 2024

Conversation

markstur
Copy link
Collaborator 

Fix to work with updated gradio

* To allow the gradio update recommended by dependabot
* launch param show_tips removed
* DataFrame for token embedding output needs a str key (columns)
* Gallery style() is gone with kwargs moved to Gallery constructor.

dependabot bot and others added 3 commits January 10, 2024 22:33
@dependabot
Bump gradio from 3.36.1 to 4.11.0
3b380f0 
Bumps [gradio](https://github.com/gradio-app/gradio) from 3.36.1 to 4.11.0.
- [Release notes](https://github.com/gradio-app/gradio/releases)
- [Changelog](https://github.com/gradio-app/gradio/blob/main/CHANGELOG.md)
- [Commits](https://github.com/gradio-app/gradio/commits)

---
updated-dependencies:
- dependency-name: gradio
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
@markstur
Fix to work with updated gradio
8391e7c 
* To allow the gradio update recommended by dependabot
* launch param show_tips removed
* DataFrame for token embedding output needs a str key (columns)
* Gallery style() is gone with kwargs moved to Gallery constructor.

Signed-off-by: markstur <[email protected]>
@markstur
Prettier formatting
d79f381 
Signed-off-by: markstur <[email protected]>
bridgetmcg
bridgetmcg approved these changes Jan 11, 2024
View reviewed changes
Copy link
Collaborator

@bridgetmcg bridgetmcg left a comment

Choose a reason for hiding this comment

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

✅

@bridgetmcg bridgetmcg merged commit 7d9ef44 into caikit:main Jan 11, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bridgetmcg bridgetmcg bridgetmcg approved these changes

@alanbraz alanbraz Awaiting requested review from alanbraz alanbraz is a code owner

@hickeyma hickeyma Awaiting requested review from hickeyma hickeyma is a code owner

@gkumbhat gkumbhat Awaiting requested review from gkumbhat gkumbhat is a code owner

@gabe-l-hart gabe-l-hart Awaiting requested review from gabe-l-hart gabe-l-hart is a code owner

@joerunde joerunde Awaiting requested review from joerunde joerunde is a code owner

@prashantgupta24 prashantgupta24 Awaiting requested review from prashantgupta24 prashantgupta24 is a code owner

@evaline-ju evaline-ju Awaiting requested review from evaline-ju evaline-ju is a code owner

@alex-jw-brooks alex-jw-brooks Awaiting requested review from alex-jw-brooks alex-jw-brooks is a code owner

@tharapalanivel tharapalanivel Awaiting requested review from tharapalanivel tharapalanivel is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@markstur @bridgetmcg