Skip to content

support customized html path and port#52

Merged
gusye1234 merged 2 commits intogusye1234:mainfrom
knightmarehs:main
Sep 19, 2024
Merged

support customized html path and port#52
gusye1234 merged 2 commits intogusye1234:mainfrom
knightmarehs:main

Conversation

@knightmarehs
Copy link
Contributor

This commit (165920c) fixed the problem about "empty graph visualization page" by saving a file graph_json.js.
While the saving path and calling path of graph_json.js in html src are both hardcode, if I html_path is not in root path, it will raise file not found error.
This pr fixed this problen and support customized html path and port.

@gusye1234
Copy link
Owner

Great works!
Can we change the comment to English? Thanks

@knightmarehs
Copy link
Contributor Author

Great works! Can we change the comment to English? Thanks

Sure, I change the comments to English
(by the way, the original comments are in Chinese so that I didn't change them in the first commit)

@gusye1234
Copy link
Owner

by the way, the original comments are in Chinese so that I didn't change them in the first commit

LGTM, and sorry about that. I should review the last PR of this file more carefully😊

@gusye1234 gusye1234 merged commit 70bbb67 into gusye1234:main Sep 19, 2024
AhmaddAbbass pushed a commit to AhmaddAbbass/nano-graphrag that referenced this pull request Nov 14, 2025
* support customized html path and port

* translate comments

---------

Co-authored-by: Sen <[email protected]>
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