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

[Backport 2.x] Add validations for Personalize input and configurations #160

Merged
merged 1 commit into from
Jul 12, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link

Backport e305c13 from #151

* Add validations for Personalize input and configurations

Description:
===========
This commit adds following changes:
* Change scoring logic to consider ranks instead of scores
* Add validation for Personalize inputs and configurations
* Add E2E tests in the form of unit tests for Personalize ranking

Signed-off-by: Ketan Kulkarni <[email protected]>

* Add user agent prefix to Personalize client to understand plugin adoption

Description:
============
This change adds following:
* User agent configuration for Personalize client to understand plugin adoption
* Add validation related unit tests

Signed-off-by: Ketan Kulkarni <[email protected]>

* Add validations for Personalize input and configurations

Description:
===========
This commit adds following changes:
* Change scoring logic to consider ranks instead of scores
* Add validation for Personalize inputs and configurations
* Add E2E tests in the form of unit tests for Personalize ranking

Signed-off-by: Ketan Kulkarni <[email protected]>

* Change response processor config name

Description:
===========
This commit includes following:
* Change response processor name from personalize_ranking to personalized_search_ranking
* Change client settings to use same naming convention
* Address review comments

Signed-off-by: Ketan Kulkarni <[email protected]>

* Fix typo in unit test

Signed-off-by: Ketan Kulkarni <[email protected]>

---------

Signed-off-by: Ketan Kulkarni <[email protected]>
(cherry picked from commit e305c13)
@mingshl mingshl merged commit 64948c5 into 2.x Jul 12, 2023
12 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants