You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I have searched the existing issues and this bug is not already filed.
My model is hosted on OpenAI or Azure. If not, please look at the "model providers" issue and don't file a new one here.
I believe this is a legitimate bug, not just a question. If this is a question, please use the Discussions area.
Describe the bug
In V1.0.0, after running the Leiden algorithm in layout_graph, some nodes were not assigned a community and level. To handle this, the code assigned pseudo community and level as a fallback mechanism. While this approach was not ideal, it did not introduce a fatal bug, as the missing nodes were still retained.
In V2.0.0, the handling of nodes changed:
Nodes are no longer saved persistently but are instead generated dynamically during queries.
The previous fallback mechanism (assigning pseudo community and level) was removed, meaning that nodes without a level attribute (level=None) are left unprocessed.
This causes an issue in _filter_under_community_level: nodes with level=None are wrongly discarded, leading to an approximately 10% data loss in the graph.
To confirm the issue, I saved nodes_df before and after _filter_under_community_level and observed that these unassigned nodes were filtered out incorrectly.
This results in significant data loss and affects query results, making it a critical bug that needs to be addressed.
Workaround
To mitigate the issue temporarily, you can add the following code snippet before the filtering step. This workaround assigns default values to missing level and community attributes, ensuring that nodes are not inadvertently discarded:
Create or load a graph that includes isolated nodes (nodes not connected to any other nodes).
Run Index
Execute local search
Observe that nodes with a missing level attribute (level=None) are discarded during the filtering process, resulting in approximately 10% of nodes being lost.
Expected Behavior
All nodes, including isolated ones, should either be assigned a valid community and level or be handled in a way that retains them in the dataset.
The filtering function should not discard nodes with a missing level attribute unless explicitly intended, thereby preventing unintended data loss.
Do you need to file an issue?
Describe the bug
In V1.0.0, after running the Leiden algorithm in
layout_graph
, some nodes were not assigned a community and level. To handle this, the code assigned pseudo community and level as a fallback mechanism. While this approach was not ideal, it did not introduce a fatal bug, as the missing nodes were still retained.In V2.0.0, the handling of nodes changed:
level=None
) are left unprocessed._filter_under_community_level
: nodes withlevel=None
are wrongly discarded, leading to an approximately 10% data loss in the graph.To confirm the issue, I saved
nodes_df
before and after_filter_under_community_level
and observed that these unassigned nodes were filtered out incorrectly.This results in significant data loss and affects query results, making it a critical bug that needs to be addressed.
Workaround
To mitigate the issue temporarily, you can add the following code snippet before the filtering step. This workaround assigns default values to missing
level
andcommunity
attributes, ensuring that nodes are not inadvertently discarded:Steps to reproduce
Expected Behavior
GraphRAG Config Used
No response
Logs and screenshots
No response
Additional Information
The text was updated successfully, but these errors were encountered: