Fix TensorFrame & make it preserve column order#126
Fix TensorFrame & make it preserve column order#126engintoklu wants to merge 1 commit intomasterfrom
Conversation
Before this commit, TensorFrame had these issues: (i) one could add a column to a TensorFrame even if the TensorFrame had a compatible number of rows with the newly added tensor; (ii) operations like adding or dropping a column resulted in non-deterministic order of columns. This commit fixes these mentioned issues.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughExtended Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #126 +/- ##
==========================================
+ Coverage 75.43% 75.61% +0.17%
==========================================
Files 59 59
Lines 9556 9611 +55
==========================================
+ Hits 7209 7267 +58
+ Misses 2347 2344 -3 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Before this pull request,
TensorFramehad these issues:This pull request fixes these mentioned issues.
Summary by CodeRabbit
New Features
Improvements