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

"edge" content as "contextual proximity" #17

Open
FJGEODEV opened this issue Dec 18, 2023 · 7 comments
Open

"edge" content as "contextual proximity" #17

FJGEODEV opened this issue Dec 18, 2023 · 7 comments

Comments

@FJGEODEV
Copy link

Thanks for your intuitive work. I have a question.

When we calculate contextual proximity, you assign

dfg2["edge"] = "contextual proximity"

When I went through every step to visualize graph, there are lots of "Edge" shown as "contextual proximity", not the content LLM generates.

Is there anything I made wrong or not configure correctly? I followed your workflow just changed the input PDF file.

Thanks.

@FJGEODEV
Copy link
Author

How can I display some content on Edge instead of word "contextual proximity"

@schiffinor
Copy link

I wrote up a fork of this that fixes this issue. Basically what happens is that edges are drawn in alphabetical order from the merged edge list. Thus if the contextual proximity edge comes after the llm edge it gets overwritten/its beneath the contextual proximity edge. I fixed this by making sure that for all llm edges, if edge A->B and B->A exist if one is a contextual proximity edge and the other is a llm edge they are merged. I also improved output a bit. However, workstation is currently out of commission as boot drive failed. Once its back up and running I'll see about uploading it as a fork or something.

@FJGEODEV
Copy link
Author

Thanks for your work, just wondering if it's good to upload the updated fork?

@schiffinor
Copy link

idrk, theres a public fork on my page with the slight edits I made. check the diffs if you'd like. Also I noticed you can use straight pdfs if you have a bunch of extra modules

@FJGEODEV
Copy link
Author

idrk, theres a public fork on my page with the slight edits I made. check the diffs if you'd like. Also I noticed you can use straight pdfs if you have a bunch of extra modules

there are multiple forks, could you point to me which one is that? Thanks.

@schiffinor
Copy link

man, I don't really know what to tell you its on my public profile under my repositories. Its a public repository that is called knowledge-graph and says forked from rahulnyk. Not much else I can do than send the link: but, you could have found this easily lol. just click on my profile. https://github.com/schiffinor/knowledge_graph

@FJGEODEV
Copy link
Author

OK, I got it. Thanks!

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

No branches or pull requests

2 participants