-
Notifications
You must be signed in to change notification settings - Fork 0
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
🏃 MetaflowRuns
#141
🏃 MetaflowRuns
#141
Conversation
Codecov Report
@@ Coverage Diff @@
## main #141 +/- ##
==========================================
+ Coverage 67.46% 67.59% +0.12%
==========================================
Files 18 18
Lines 707 719 +12
==========================================
+ Hits 477 486 +9
- Misses 230 233 +3
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
d7e8f61
to
93e4461
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.
👍 with caveat that we'll probably want to switch Batch Detail view back to using a media file view, so that it will display media files for which there are not yet any Metaflow Runs, but we can merge this now and build off it.
MetaflowRuns
Adds fields to
MetaflowRun
table for caching results in Chowda dbfinished: bool
successful: bool
created_at: datetime
utcnow
finished_at: datetime
BatchMetaflowRunDisplayField
Migrates
BatchMediaFileDisplayField
->BatchMetaflowRunDisplayField
Closes #67