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

feat: Add similarity threshold to amenity search #477

Merged
merged 16 commits into from
Sep 4, 2024
Merged

Conversation

ferdeleong
Copy link
Contributor

@ferdeleong ferdeleong commented Aug 23, 2024

Add score to results from vector similarity search and filter by similarity threshold.

Limit vector similarity search results to top k to limit the graph retrieved nodes readability.

Return SIMILAR_TO results in a collection (list) to avoid details from the source node for each retrieved node.

Cleaning csv file from duplicated relationships and add unidentified LLM relationships.

@ferdeleong ferdeleong self-assigned this Aug 23, 2024
@ferdeleong ferdeleong requested a review from duwenxin99 August 26, 2024 21:19
@ferdeleong ferdeleong requested a review from averikitsch August 26, 2024 21:31
@@ -171,48 +171,58 @@ async def test_total_similar_to_relationships_count(ds: neo4j_graph.Client):
print(count)

# Bi-directional pairs are counted only once
expected_count = 217
expected_count = 172
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I polished relationships.csv file, causing the deletion of some irrelevant relationships

@ferdeleong ferdeleong marked this pull request as ready for review September 4, 2024 05:59
@ferdeleong ferdeleong requested a review from a team as a code owner September 4, 2024 05:59
@ferdeleong ferdeleong force-pushed the neo4j-service-12 branch 2 times, most recently from 650fedb to b110013 Compare September 4, 2024 06:33
@ferdeleong ferdeleong merged commit c49bef9 into main Sep 4, 2024
18 checks passed
@ferdeleong ferdeleong deleted the neo4j-service-12 branch September 4, 2024 06:40
anubhav756 pushed a commit that referenced this pull request Dec 3, 2024
🤖 I have created a release *beep* *boop*
---


##
[0.3.0](v0.2.0...v0.3.0)
(2024-12-03)


### Features

* Add similarity threshold to amenity search
([#477](#477))
([c49bef9](c49bef9))
* Add tracing for AlloyDB and CloudSQL Postgres providers
([#494](#494))
([2fa03bc](2fa03bc))
* Consolidate postgres providers
([#493](#493))
([a3b2c42](a3b2c42))
* Reuse connector object across different database connections in…
([#487](#487))
([61c0f52](61c0f52))
* Switch the llm to ChatVertexAI
([#486](#486))
([479c5e5](479c5e5))


### Bug Fixes

* Add test cases to improve coverage for postgres and over more tools.
([#508](#508))
([20870ea](20870ea))
* Reuse connector object across different database connections.
([#484](#484))
([2b05739](2b05739)),
closes
[#416](#416)
* update close client function to async
([#505](#505))
([b614276](b614276))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

3 participants