feat: support dynacmic graph via neo4j #1254
Conversation
NeilJohnson0930
force-pushed
the
dev_dynamic
branch
from
500b659 to
f141136
Compare
|
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
NeilJohnson0930
changed the title
NeilJohnson0930
force-pushed
the
dev_dynamic
branch
from
f141136 to
fac6696
Compare
|
Got it, thx @keli-wen |
NeilJohnson0930
force-pushed
the
dev_dynamic
branch
from
7a2e1b1 to
c2e2405
Compare
NeilJohnson0930
requested review from
raywhoelse and
Wendong-Fan
and removed request for
raywhoelse and
Wendong-Fan
|
View / edit / reply to this conversation on ReviewNB Wendong-Fan commented on 2025-02-19T09:59:45Z for the title could we highlight the finance use case a little bit? NeilJohnson0930 commented on 2025-02-19T12:26:27Z Dynamic Knowledge Graph Construction for Financial Report with CAMEL |
|
View / edit / reply to this conversation on ReviewNB Wendong-Fan commented on 2025-02-19T09:59:46Z use camel's latest version, don't install [all], only install necessary dependency like [rag] NeilJohnson0930 commented on 2025-02-19T12:27:27Z pip install "camel-ai[rag]==0.2.22" |
|
View / edit / reply to this conversation on ReviewNB Wendong-Fan commented on 2025-02-19T09:59:47Z the api key setting part is different with other cookbooks, refer to https://colab.research.google.com/drive/17SiWWjoK7l8Sy9FBsGKUHC6zuEsLt2yX?usp=sharing NeilJohnson0930 commented on 2025-02-25T03:21:59Z API key setting has been changed to require user input via getpass |
|
View / edit / reply to this conversation on ReviewNB Wendong-Fan commented on 2025-02-19T09:59:47Z no need to print this NeilJohnson0930 commented on 2025-02-25T03:22:43Z Removed |
|
View / edit / reply to this conversation on ReviewNB Wendong-Fan commented on 2025-02-19T09:59:48Z Line #27. # Set up Mistral Large 2 model no need to use mistral model in this cookbook, could we use together ai instead? since we can also announce this cookbook with them https://www.together.ai/ NeilJohnson0930 commented on 2025-02-25T03:24:58Z Latest cookbook has been updated to replace the Mistral model with the Together model. |
|
View / edit / reply to this conversation on ReviewNB Wendong-Fan commented on 2025-02-19T09:59:49Z Line #46. model_type=ModelType.GPT_3_5_TURBO, why here is gpt 35 turbo, I think use gpt 4o mini would be more suitable, since o1 would make high cost and slow NeilJohnson0930 commented on 2025-02-25T03:26:22Z GPT-4o Mini model has been set |
|
View / edit / reply to this conversation on ReviewNB Wendong-Fan commented on 2025-02-19T09:59:49Z this code chunk is too big and including different parts, could we splite it to smaller code chunk and add more description to make it more easy to read? NeilJohnson0930 commented on 2025-02-25T03:30:02Z Thanks for the suggestion! I've broken the code into smaller chunks and added more descriptions to enhance readability. Let me know if you have any further feedback! |
|
View / edit / reply to this conversation on ReviewNB Wendong-Fan commented on 2025-02-19T09:59:50Z "access results by navigating to http://localhost:7474 in your web browser" is the guide for local deploy, for user using neo4j cloud, we also need to provide link to cloud db NeilJohnson0930 commented on 2025-02-25T03:35:29Z Thanks for the feedback! I've updated the guide to generate the graph in Neo4j Aura for cloud deployment and included screenshots/GIFs to showcase the graph. However, since enterprise account features are not supported at the moment, sharing a direct graph link will be delayed. |
There was a problem hiding this comment.
thanks @NeilJohnson0930 and @keli-wen , left some comments for both the feature and cookbook
Dynamic Knowledge Graph Construction for Financial Report with CAMELView entire conversation on ReviewNB |
|
pip install "camel-ai[rag]==0.2.22" View entire conversation on ReviewNB |
|
API key setting has been changed to require user input via getpass View entire conversation on ReviewNB |
|
Removed View entire conversation on ReviewNB |
|
Latest cookbook has been updated to replace the Mistral model with the Together model. View entire conversation on ReviewNB |
|
GPT-4o Mini model has been set View entire conversation on ReviewNB |
|
Thanks for the suggestion! I've broken the code into smaller chunks and added more descriptions to enhance readability. Let me know if you have any further feedback! View entire conversation on ReviewNB |
|
Thanks for the feedback! I've updated the guide to generate the graph in Neo4j Aura for cloud deployment and included screenshots/GIFs to showcase the graph. However, since enterprise account features are not supported at the moment, sharing a direct graph link will be delayed. View entire conversation on ReviewNB |
Description
Describe your changes in detail.
Motivation and Context
Why is this change required? What problem does it solve?
If it fixes an open issue, please link to the issue here.
You can use the syntax
close #15213if this solves the issue #15213Types of changes
What types of changes does your code introduce? Put an
xin all the boxes that apply:Implemented Tasks
Checklist
Go over all the following points, and put an
xin all the boxes that apply.If you are unsure about any of these, don't hesitate to ask. We are here to help!