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

Update requirements.txt #10

Closed
wants to merge 2 commits into from
Closed

Update requirements.txt #10

wants to merge 2 commits into from

Conversation

wiljav
Copy link

@wiljav wiljav commented Mar 16, 2024

Changed torch and gradio versions to install latest version instead on one specific version.

The previous version caused issues related to dependency conflicts, example here:

Changed 'torch' and 'gradio' versions to install latest version instead on one specific version
@TonyLianLong
Copy link
Owner

Have you tested the demo and made sure the style (CSS) does not break? If so, I'll merge the PR.

Added 'peft' and changed back 'gradio' to the old version 3.14.0, because there are some outdated syntax within app.py that the new gradio version (> 4.0) doesn't recognise them.
@wiljav
Copy link
Author

wiljav commented Mar 16, 2024

I added peft and changed back gradio to version 3.14.0, the new version (>4.0) doesn't support some deprecated syntax (https://www.gradio.app/changelog#4-21-0) in app.py, which caused issues.
Tested and works as expected now.

@wiljav wiljav closed this by deleting the head repository Apr 6, 2024
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