Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #645 +/- ##
==========================================
- Coverage 86.49% 86.48% -0.01%
==========================================
Files 42 42
Lines 5160 5159 -1
==========================================
- Hits 4463 4462 -1
Misses 697 697
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
| modality: single_table | ||
| type: Deep Learning |
There was a problem hiding this comment.
On the website, we use type to describe the modality key, while type itself refers to something else. We may want to double-check this.
There was a problem hiding this comment.
@R-Palazzo the website will pull the modality from this sheet that's generated during the benchmark and it labels it with "Type". The fact that we use type to describe the method is confusing, however, we should keep it as is for now.
One thing that I'm worried about is why TabDDPM was never added to the sheet, but ClavaDDPM was. I think there might be something I missed.
Resolve #644