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

chore: use structured output in Kyma RAG reranker #308

Merged

Conversation

marcobebway
Copy link
Collaborator

@marcobebway marcobebway commented Dec 10, 2024

Description

Use structured output in Kyma RAG reranker.

@marcobebway marcobebway requested a review from a team as a code owner December 10, 2024 16:19
Copy link

Note(s) for PR Auther:

  • The integration test will be skipped for the PR. You can trigger it manually after adding the label: run-integration-test.
  • The evaluation test will be skipped for the PR. You can trigger it manually after adding the label: evaluation requested.
  • If any changes are made to the evaluation tests data, make sure that the integration tests are working as expected.
  • If any changes are made to how to run the unit tests, make sure to update the steps for unit-tests in the create-release.yml workflow as well.

Note(s) for PR Reviewer(s):

  • Make sure that the integration and evaluation tests are working as expected.

@marcobebway marcobebway self-assigned this Dec 10, 2024
@marcobebway marcobebway force-pushed the use-structured-output-in-reranker branch from 6b656f4 to 1d2b396 Compare December 10, 2024 16:23
@marcobebway marcobebway added evaluation requested A PR with this label will trigger the validation workflow. run-integration-test labels Dec 10, 2024
@marcobebway
Copy link
Collaborator Author

/hold

@marcobebway marcobebway force-pushed the use-structured-output-in-reranker branch 2 times, most recently from 14b69c4 to 0bd658a Compare December 11, 2024 10:49
@marcobebway marcobebway force-pushed the use-structured-output-in-reranker branch from 0bd658a to 28a82cd Compare December 11, 2024 10:53
@marcobebway
Copy link
Collaborator Author

/hold cancel

@kyma-bot kyma-bot added the lgtm label Dec 11, 2024
@kyma-bot kyma-bot merged commit 5d4e01a into kyma-project:main Dec 11, 2024
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes evaluation requested A PR with this label will trigger the validation workflow. lgtm run-integration-test size/S
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants