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

Hardcoded column name in DocumentSimilarityRanker annotator #14195

Closed
1 task done
jfernandrezj opened this issue Mar 5, 2024 · 0 comments · Fixed by #14381
Closed
1 task done

Hardcoded column name in DocumentSimilarityRanker annotator #14195

jfernandrezj opened this issue Mar 5, 2024 · 0 comments · Fixed by #14381
Assignees
Labels

Comments

@jfernandrezj
Copy link
Contributor

Is there an existing issue for this?

  • I have searched the existing issues and did not find a match.

Who can help?

@danilojsl @maziyarpanahi

What are you working on?

Getting similar documents from a knowledge base for RAG applications

Current Behavior

Both current behavior and desired behavior is documented in the branch with the failing tests:
https://github.com/JohnSnowLabs/spark-nlp/tree/issues/document-similarity-ranker-failing-test

Expected Behavior

Both current behavior and desired behavior is documented in the branch with the failing tests:
https://github.com/JohnSnowLabs/spark-nlp/tree/issues/document-similarity-ranker-failing-test

Steps To Reproduce

Just run the added tests in the branch:
https://github.com/JohnSnowLabs/spark-nlp/tree/issues/document-similarity-ranker-failing-test

Spark NLP version and Apache Spark

Spark 3.4
Spark NLP 5.2.2

Type of Spark Application

No response

Java Version

Java 11

Java Home Directory

No response

Setup and installation

No response

Operating System and Version

No response

Link to your project (if available)

No response

Additional Information

No response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants