Skip to content

Conversation

so-engineer
Copy link

Since the zip function adopts the length of the shorter iterable, information loss can occur.
Therefore, it is necessary to ensure that the number of embeddings matches the number of node_id_mapping.
I have made the necessary changes. Could you please review them at your convenience?

Steps to reproduce

python graphrag.py

Actual result

Ingesting to Qdrant...
len(embeddings):  25
len(node_id_mapping):  21  ←It needs to be the same as the number of embeddings.
Qdrant ingestion complete

Expected result

Ingesting to Qdrant...
len(embeddings):  25
len(node_id_mapping_list):  25
Qdrant ingestion complete

@so-engineer so-engineer marked this pull request as draft April 12, 2025 21:52
@so-engineer so-engineer marked this pull request as ready for review April 12, 2025 21:52
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.

1 participant