-
Notifications
You must be signed in to change notification settings - Fork 27
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 flatten vector. Stackup columnwise instead of row wise #345
Conversation
Signed-off-by: ssrigiri1 <shashank_srigiri@intuit.com>
aecc1ec
to
ffd84a0
Compare
Signed-off-by: ssrigiri1 <shashank_srigiri@intuit.com>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #345 +/- ##
==========================================
+ Coverage 93.40% 93.56% +0.15%
==========================================
Files 83 83
Lines 3853 3853
Branches 311 311
==========================================
+ Hits 3599 3605 +6
+ Misses 197 193 -4
+ Partials 57 55 -2 ☔ View full report in Codecov by Sentry. |
Can you make this parameter driven? |
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.
@shashank10456 can you add unit test and assert the expected result?
Signed-off-by: ssrigiri1 <shashank_srigiri@intuit.com>
Signed-off-by: ssrigiri1 <shashank_srigiri@intuit.com>
Signed-off-by: ssrigiri1 <shashank_srigiri@intuit.com>
Signed-off-by: ssrigiri1 <shashank_srigiri@intuit.com>
Signed-off-by: ssrigiri1 <shashank_srigiri@intuit.com>
Signed-off-by: ssrigiri1 <shashank_srigiri@intuit.com>
Explain what this PR does.
Fix flatten vector. Stackup columnwise instead of row wise