Update constraints to be compatible with composite keys defined in the metadata#2828
Conversation
7d1df4f to
ff2d517
Compare
fealho
left a comment
There was a problem hiding this comment.
The _remove_columns_from_metadata method in sdv.cag._utils doesn't support composite keys. Should that be addressed in this PR?
|
This Pull Request is not linked to an issue. To ensure our community is able to accurately track resolved issues, please link any issue that will be closed by this PR! |
fdc2f9d to
799c9ee
Compare
cef7bc9 to
e79c4fa
Compare
|
This Pull Request is not linked to an issue. To ensure our community is able to accurately track resolved issues, please link any issue that will be closed by this PR! |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## feature/metadata-composite-keys #2828 +/- ##
====================================================================
+ Coverage 75.41% 98.06% +22.64%
====================================================================
Files 73 73
Lines 8201 8266 +65
====================================================================
+ Hits 6185 8106 +1921
+ Misses 2016 160 -1856
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
This Pull Request is not linked to an issue. To ensure our community is able to accurately track resolved issues, please link any issue that will be closed by this PR! |
Resolve datacebo/SDV-Enterprise#1735