Force to use dedupe int ID from FACT INVALIDATION CANDIDATES #885
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
I'm parsing documentation and code with graphiti, I wrote a Bedrock (aws) adapter to use with it, (Maybe that's why I needed to change the message) and in some cases the documentation/code is repeated and the deduplication process fails because Bedrock (Nova) returns the uuid from the entity instead of the item index from the facts candidates. Checking the dedupe code, we need always the index instead of the ID.
Adding this extra sentence fixed the issue for me.
(The issue is when deduplicating edges)
Type of Change
The model input message:
Model output:
Testing
(Would be great if somebody could test this for other models)
Breaking Changes
Checklist
make lint
passes)