Closed bjwswang closed 1 year ago
Merging #324 (8406829) into main (785e857) will decrease coverage by
2.61%
. The diff coverage is10.73%
.
@@ Coverage Diff @@
## main #324 +/- ##
==========================================
- Coverage 25.12% 22.51% -2.61%
==========================================
Files 36 36
Lines 3240 3264 +24
==========================================
- Hits 814 735 -79
- Misses 2374 2491 +117
+ Partials 52 38 -14
Files Changed | Coverage Δ | |
---|---|---|
api/v1alpha1/rating_types.go | 100.00% <ø> (ø) |
|
controllers/rating_controller.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
@0xff-dev any other comments?
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 keyTo use this
Evaluator
in Rating,user must install kubeagi/arcadia as well.