Closed bjwswang closed 1 year ago
Merging #319 (d0c6cfc) into main (67b36b1) will decrease coverage by
3.63%
. The diff coverage is5.81%
.:exclamation: Current head d0c6cfc differs from pull request most recent head 9c9ef32. Consider uploading reports for the commit 9c9ef32 to get more accurate results
@@ Coverage Diff @@
## main #319 +/- ##
==========================================
- Coverage 25.31% 21.68% -3.63%
==========================================
Files 35 36 +1
Lines 3216 3390 +174
==========================================
- Hits 814 735 -79
- Misses 2350 2617 +267
+ Partials 52 38 -14
Files Changed | Coverage Δ | |
---|---|---|
api/v1alpha1/rating_types.go | 100.00% <ø> (ø) |
|
controllers/rating_controller.go | 0.00% <0.00%> (ø) |
|
pkg/evaluator/evaluator.go | 0.00% <0.00%> (ø) |
|
api/v1alpha1/rating.go | 95.32% <86.36%> (+16.47%) |
:arrow_up: |
... and 1 file with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
What type of PR is this?
/kind feat
What this PR does / why we need it
Which issue(s) this PR fixes
Fixes #285 Fixes #286
Special notes for your reviewer
This PR enables AI Evaluator for Rating which includes below changes:
Evaluator
in rating spec to configure AI LLM which shall be used in this RatingDimension
in PipelineParam to make sure only onedimesion
pipelinerun exists.dimension
as the key inrating.status.pipelineruns
pkg/evaluator
to implement aevaluator agent
evaluator agent
to evaluate this rating dimesion with the help ofkubeagi/arcadia
rating.status.evaluations with
dimesion` as the key