Skip to content

Rhoaieng 79828 implement dspy based prompt optimization - #117

Open
filip-komarzyniec wants to merge 7 commits into
mainfrom
RHOAIENG-79828-implement-dspy-based-prompt-optimization
Open

filip-komarzyniec wants to merge 7 commits into
mainfrom
RHOAIENG-79828-implement-dspy-based-prompt-optimization

Conversation

@filip-komarzyniec

Copy link
Copy Markdown
Collaborator

Description

Introducing optional DSPy-based prompts optimization.

Motivation

Increase quality of the end results and enhance prompt's maintainability.

Changes

  • new dependency of dspy
  • modified KFP AutoRAG pipeline's components so that they can trigger DSPy-based prompts optimization
  • modified ai4rag's core source files in order to enable prompt optimization:
    • new module ai4rag.core.hpo.prompts
    • modified AI4RAGExperiment, ModelPreSelector and SimpleRag classes and their internal flow

Testing

yes

Checklist

  • Tests added/updated
  • Documentation updated
  • Code follows style guide
  • All checks passing

Signed-off-by: Filip Komarzyniec <fkomarzy@redhat.com>

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED
…pipeline

Signed-off-by: Filip Komarzyniec <fkomarzy@redhat.com>

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED
Signed-off-by: Filip Komarzyniec <fkomarzy@redhat.com>

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED
Signed-off-by: Filip Komarzyniec <fkomarzy@redhat.com>

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED
…utoRAG pipeline

Signed-off-by: Filip Komarzyniec <fkomarzy@redhat.com>

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED
Signed-off-by: Filip Komarzyniec <fkomarzy@redhat.com>

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED
…f KFP AutoRAG pipeline

Signed-off-by: Filip Komarzyniec <fkomarzy@redhat.com>

rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED
@jakub-walaszczyk

Copy link
Copy Markdown
Collaborator

@filip-komarzyniec is it still valid? Can we close it?

@filip-komarzyniec

Copy link
Copy Markdown
Collaborator Author

@jakub-walaszczyk The branch definitely needs to stay there. The feature is currently put on hold, but not abandoned.

The PR, if you want to, can be closed for the time being as it serves only display purposes.

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.

2 participants