haystack icon indicating copy to clipboard operation
haystack copied to clipboard

feat: allow `Generators` to run with a system prompt defined at run time

Open davidsbatista opened this issue 1 year ago • 3 comments

Related Issues

  • fixes #issue-number

Proposed Changes:

How did you test it?

Notes for the reviewer

Checklist

davidsbatista avatar Sep 30 '24 13:09 davidsbatista

Pull Request Test Coverage Report for Build 11443425066

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 3 unchanged lines in 1 file lost coverage.
  • Overall coverage decreased (-0.005%) to 90.344%

Files with Coverage Reduction New Missed Lines %
components/generators/openai.py 3 95.56%
<!-- Total: 3
Totals Coverage Status
Change from base Build 11435566584: -0.005%
Covered Lines: 7504
Relevant Lines: 8306

💛 - Coveralls

coveralls avatar Sep 30 '24 13:09 coveralls

Changed the code support the 3 uses cases without loss of information:

  • no system prompt
  • using a system prompt defined at init
  • using a system prompt defined at run time

davidsbatista avatar Oct 07 '24 12:10 davidsbatista

@davidsbatista looks good!

sjrl avatar Oct 10 '24 09:10 sjrl