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 FE from 4.x -> latest (5.xx) & check compatibility carefully #173

Open
imbajin opened this issue Feb 21, 2025 · 17 comments · May be fixed by #197
Open

Update FE from 4.x -> latest (5.xx) & check compatibility carefully #173

imbajin opened this issue Feb 21, 2025 · 17 comments · May be fixed by #197
Assignees
Labels
dependencies Pull requests that update a dependency file

Comments

@imbajin
Copy link
Member

imbajin commented Feb 21, 2025

update Gradio from 4.x -> latest (5.xx) & check compatibility carefully

@imbajin imbajin changed the title update [Gradio](https://github.com/gradio-app/gradio) from 4.x -> latest (5.xx) & check compatibility carefully Update FE from 4.x -> latest (5.xx) & check compatibility carefully Feb 21, 2025
@dosubot dosubot bot added the dependencies Pull requests that update a dependency file label Feb 21, 2025
@Aryankb
Copy link
Contributor

Aryankb commented Feb 24, 2025

update Gradio from 4.x -> latest (5.xx) & check compatibility carefully

@imbajin i am working on this

@imbajin
Copy link
Member Author

imbajin commented Feb 24, 2025

update Gradio from 4.x -> latest (5.xx) & check compatibility carefully

@imbajin i am working on this

@Aryankb THX, some points to note:

  1. We need better component support for our streaming output conversations, in 4.x it's not good (Current effect could refer feat(llm): add streaming output feature #172)
  2. It's been almost half a year now, what are the major updates/performance improvements in this area?
  3. Keep an eye on the official "Breaking Change" & confirm if we have any related issues

@imbajin imbajin moved this to 🏗 In progress in HugeGraph Tasks Feb 24, 2025
@imbajin
Copy link
Member Author

imbajin commented Mar 4, 2025

update Gradio from 4.x -> latest (5.xx) & check compatibility carefully

@imbajin i am working on this

@Aryankb Excuse me, any progress on this mission?

Because we are currently undergoing a workflow transformation and preparing to introduce a workflow framework, we would like to complete the upgrade of Gradio's major version first to reduce potential conflicts/security issues in the future

@Aryankb
Copy link
Contributor

Aryankb commented Mar 4, 2025

@imbajin ok sir. sorry for the delay. Will work on it today. I was busy in my college projects and intern works

@Aryankb
Copy link
Contributor

Aryankb commented Mar 4, 2025

@imbajin just wanted to share and know if this gradio update helpful? it happened just 4 days ago and mentions something about streaming

Image
pls refer :- gradio-app/gradio#10688

also can you please guide me how can i test streaming?

@imbajin
Copy link
Member Author

imbajin commented Mar 5, 2025

@imbajin just wanted to share and know if this gradio update helpful? it happened just 4 days ago and mentions something about streaming

Image pls refer :- gradio-app/gradio#10688

also can you please guide me how can i test streaming?

Thanks for your info, maybe we do need test to ensure it

Image

After #190 merged? (or u could use that PR's branch)

@Aryankb
Copy link
Contributor

Aryankb commented Mar 5, 2025

@imbajin should i use #190 PR's branch? and test it. also how can i test it? can i test directly from ui? if yes which buttons should i click? can you please give a basic overview of this? also i am reading wiki to know all features

@imbajin
Copy link
Member Author

imbajin commented Mar 6, 2025

@imbajin should i use #190 PR's branch? and test it. also how can i test it? can i test directly from ui? if yes which buttons should i click? can you please give a basic overview of this? also i am reading wiki to know all features

Yes, u're right, clicking the answer button in UI is enough~

Image

Can you please give a basic overview of this?

@MrJs133 maybe add the "Quick Start" operation doc in Wiki first?

@MrJs133 MrJs133 closed this as completed Mar 6, 2025
@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in HugeGraph Tasks Mar 6, 2025
@MrJs133 MrJs133 reopened this Mar 6, 2025
@MrJs133
Copy link
Collaborator

MrJs133 commented Mar 6, 2025

@imbajin should i use #190 PR's branch? and test it. also how can i test it? can i test directly from ui? if yes which buttons should i click? can you please give a basic overview of this? also i am reading wiki to know all features

Yes, u're right, clicking the answer button in UI is enough~

Image > Can you please give a basic overview of this?

@MrJs133 maybe add the "Quick Start" operation doc in Wiki first?

OK I will do it today (Sorry, I accidentally clicked theclose with comment button)

@imbajin
Copy link
Member Author

imbajin commented Mar 6, 2025

@imbajin should i use #190 PR's branch? and test it. also how can i test it? can i test directly from ui? if yes which buttons should i click? can you please give a basic overview of this? also i am reading wiki to know all features

@Aryankb Already merged into main/master branch (could test it directly)

@Aryankb
Copy link
Contributor

Aryankb commented Mar 6, 2025

@imbajin ok sir, thanks. Just wanted to know, what all things i need to configure before that button click?

  • fill open ai api key (can i control the token usage?)
  • any more steps ?

facing the following error on clicking that button:-

Image

@imbajin
Copy link
Member Author

imbajin commented Mar 6, 2025

  1. fill open ai api key (can i control the token usage?)
  2. any more steps ?
  1. Yes, u could control it here & could also test it only in raw answer first

Image

BTW, no need to use OpenAI key for testing, you can use any free model completely (By LiteLLM option, see it in README)

  1. Nope, but it's best to test the situation where both options are checked at the same time, as this will result in parallel output
Image

@Aryankb
Copy link
Contributor

Aryankb commented Mar 6, 2025

@imbajin facing this error on button click

facing the following error on clicking that button:-

Image

i think the error is related to hugegraph server configuration

@imbajin
Copy link
Member Author

imbajin commented Mar 6, 2025

i think the error is related to hugegraph server configuration

Yes, u need to follow the README's instruction to run a graph server first & config it (apply the configs)

https://github.com/apache/incubator-hugegraph-ai/tree/main/hugegraph-llm#2-environment-requirements

@MrJs133 will add the basic UI instruction soon~

@Aryankb
Copy link
Contributor

Aryankb commented Mar 6, 2025

Yes, u need to follow the README's instruction to run a graph server first & config it (apply the configs)

Yes i did everything. Server is running on docker

@Aryankb
Copy link
Contributor

Aryankb commented Mar 6, 2025

it worked. actually it's required to load into graph first. then only hugegraph RAG query works

@imbajin
Copy link
Member Author

imbajin commented Mar 7, 2025

i think the error is related to hugegraph server configuration

Yes, u need to follow the README's instruction to run a graph server first & config it (apply the configs)

main/hugegraph-llm#2-environment-requirements

@MrJs133 will add the basic UI instruction soon~

@Aryankb Already add a basic doc for the UI & logic (U could enhance it later~)

refer https://github.com/apache/incubator-hugegraph-ai/blob/main/hugegraph-llm/quick_start.md

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
Status: Done
3 participants