-
Notifications
You must be signed in to change notification settings - Fork 0
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
Adding ability to run query sets and save results on OpenSearch #49
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
First glance over the code looks very good!
I left a couple of comments that I'm happy to talk about in more depth.
I'll run the defined endpoints also locally and add any findings to this review.
...-evaluation-plugin/src/main/java/org/opensearch/eval/SearchQualityEvaluationRestHandler.java
Outdated
Show resolved
Hide resolved
...ty-evaluation-plugin/src/main/java/org/opensearch/eval/runners/OpenSearchQuerySetRunner.java
Outdated
Show resolved
Hide resolved
...ty-evaluation-plugin/src/main/java/org/opensearch/eval/runners/OpenSearchQuerySetRunner.java
Outdated
Show resolved
Hide resolved
...ty-evaluation-plugin/src/main/java/org/opensearch/eval/runners/OpenSearchQuerySetRunner.java
Outdated
Show resolved
Hide resolved
...earch-quality-evaluation-plugin/src/main/java/org/opensearch/eval/runners/SearchMetrics.java
Show resolved
Hide resolved
* @param judgments A list of {@link Judgment judgments} used for metric calculation. | ||
* @param k The k used for metrics calculation, i.e. DCG@k. | ||
*/ | ||
public SearchMetrics(final List<QueryResult> queryResults, final List<Judgment> judgments, final int k) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I plan to abstract out these metrics, i.e. SearchMetrics
with implementations like DcgSearchMetrics
and NdcgSearchMetrics
to let others add search metrics.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work, this looks really great!
Thanks for working on my feedback so quickly
No description provided.