merlin icon indicating copy to clipboard operation
merlin copied to clipboard

feat(mlflow): Upgrade mlflow versions

Open deadlycoconuts opened this issue 1 year ago • 1 comments

Description

Modifications

Tests

Checklist

  • [ ] Added PR label
  • [ ] Added unit test, integration, and/or e2e tests
  • [ ] Tested locally
  • [ ] Updated documentation
  • [ ] Update Swagger spec if the PR introduce API changes
  • [ ] Regenerated Golang and Python client if the PR introduces API changes

Release Notes


deadlycoconuts avatar Oct 16 '24 05:10 deadlycoconuts

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 59.67%. Comparing base (0877a4f) to head (8931222).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #614      +/-   ##
==========================================
- Coverage   60.68%   59.67%   -1.02%     
==========================================
  Files         277      277              
  Lines       22245    26006    +3761     
==========================================
+ Hits        13500    15519    +2019     
- Misses       7877     9619    +1742     
  Partials      868      868              
Flag Coverage Δ
api-test 57.86% <ø> (-0.81%) :arrow_down:
sdk-test-3.10 75.51% <ø> (ø)
sdk-test-3.8 75.49% <ø> (ø)
sdk-test-3.9 75.49% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Oct 16 '24 05:10 codecov[bot]

do we still need this?

tiopramayudi avatar Nov 21 '24 03:11 tiopramayudi

Oh good point, no we don't I'll close this PR. Thanks for pointing this out.

deadlycoconuts avatar Nov 21 '24 03:11 deadlycoconuts