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

add text2cypher component #1319

Open
wants to merge 129 commits into
base: main
Choose a base branch
from

Conversation

jeanyu-habana
Copy link

Description

Implement Cypher Query based on input text**: Transform user-provided natural language into Cypher queries, subsequently executing them to retrieve data from Graph databases.

Issues

n/a

Type of change

List the type of change like below. Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds new functionality)
  • Breaking change (fix or feature that would break existing design and interface)
  • Others (enhancement, documentation, validation, etc.)

Dependencies

neo4j microservice

Tests

user-provided natural language transformed into Cypher queries, subsequently executing them to retrieve data from Neo4J database.

Signed-off-by: jeanyu-habana <[email protected]>
@chensuyue
Copy link
Collaborator

Please check and fix the CI issues.


args.load_quantized_model = False
args.num_return_sequences = 1
args.model_name_or_path = "neo4j/text2cypher-gemma-2-9b-it-finetuned-2024v1"
Copy link
Member

Choose a reason for hiding this comment

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

I see many of your code contains deepspeed and distributed model initializing steps, but here it seems this 9b model is the only option, that does not require a deepspeed setting. Have you tried the neo4j/neo4j_llama318b_finetuned_merged_oct24 which use a 318b model? If not I guess you can remove the deepspeed related stuff for now.

Copy link
Author

Choose a reason for hiding this comment

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

done.

Copy link
Author

Choose a reason for hiding this comment

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

done. deepspeed/multi-device code was removed.

Copy link
Member

@Spycsh Spycsh left a comment

Choose a reason for hiding this comment

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

Please remove/add some content to the empty readme under text2cypher/deployment/docker_compose. Please also change the title of this PR (e.g. add text2cypher component).

Signed-off-by: jeanyu-habana <[email protected]>
Signed-off-by: jeanyu-habana <[email protected]>
Signed-off-by: jeanyu-habana <[email protected]>
Signed-off-by: jeanyu-habana <[email protected]>
@jeanyu-habana jeanyu-habana changed the title initial commit add text2cypher component Feb 25, 2025
@jeanyu-habana
Copy link
Author

Please remove/add some content to the empty readme under text2cypher/deployment/docker_compose. Please also change the title of this PR (e.g. add text2cypher component).

Thank you for your comment. README updated and PR title changed.

pre-commit-ci bot and others added 30 commits March 2, 2025 20:37
Signed-off-by: jeanyu-habana <[email protected]>
Signed-off-by: jeanyu-habana <[email protected]>
Signed-off-by: jeanyu-habana <[email protected]>
Signed-off-by: jeanyu-habana <[email protected]>
Signed-off-by: jeanyu-habana <[email protected]>
Signed-off-by: jeanyu-habana <[email protected]>
Signed-off-by: jeanyu-habana <[email protected]>
Signed-off-by: jeanyu-habana <[email protected]>
Signed-off-by: jeanyu-habana <[email protected]>
Signed-off-by: jeanyu-habana <[email protected]>
Signed-off-by: jeanyu-habana <[email protected]>
Signed-off-by: jeanyu-habana <[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.

3 participants