-
Notifications
You must be signed in to change notification settings - Fork 357
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
Groupby.nunique #617
Groupby.nunique #617
Conversation
Codecov Report
@@ Coverage Diff @@
## master #617 +/- ##
==========================================
+ Coverage 92.97% 92.99% +0.01%
==========================================
Files 31 31
Lines 5085 5094 +9
==========================================
+ Hits 4728 4737 +9
Misses 357 357
Continue to review full report at Codecov.
|
@RainFung, can you resolve the conflicts? looks close to go. |
Argh, this conflict was caused by merging #622. Can you update please? |
@RainFung, I will merge this one as soon as the tests pass |
Can you solve this before merge?#617 (comment) |
Softagram Impact Report for pull/617 (head commit: ad99a8e)⭐ Change Overview
📄 Full report
Give feedback on this report to support@softagram.com |
wait to be code reviewed. |
add parameter to check if all columns to be agged