-
Notifications
You must be signed in to change notification settings - Fork 362
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: fix master config and experiment config for log retention #9075
Conversation
✅ Deploy Preview for determined-ui ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
ef40754
to
212f05e
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #9075 +/- ##
==========================================
- Coverage 47.06% 42.80% -4.27%
==========================================
Files 1154 710 -444
Lines 142378 100529 -41849
Branches 2423 2423
==========================================
- Hits 67006 43027 -23979
+ Misses 75182 57312 -17870
Partials 190 190
Flags with carried forward coverage won't be shown. Click here to find out more. |
f21075e
to
73904e6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
suggested edits
73904e6
to
7836c9b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
model dev stamp
7836c9b
to
212c6bb
Compare
212c6bb
to
c9585b4
Compare
Description
Fixing the experiment and master config to make it consistent and adding a wrapper layer.
Test Plan
Make sure all tests pass.
Commentary (optional)
Checklist
docs/release-notes/
.See Release Note for details.
Ticket