-
Notifications
You must be signed in to change notification settings - Fork 651
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-#1869: index sort for count(level=...) #1870
Conversation
is there are any other operations that uses |
For now |
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.
LGTM then
Codecov Report
@@ Coverage Diff @@
## master #1870 +/- ##
==========================================
+ Coverage 81.39% 81.40% +0.01%
==========================================
Files 79 79
Lines 9165 9165
==========================================
+ Hits 7460 7461 +1
+ Misses 1705 1704 -1
Continue to review full report at Codecov.
|
Signed-off-by: Alexander Myskov <alexander.myskov@intel.com>
Signed-off-by: Alexander Myskov <alexander.myskov@intel.com>
Signed-off-by: Alexander Myskov <alexander.myskov@intel.com>
@dchigarev your approve was dismissed during rebase, can you make approve one more time? |
…t#1870) * FIX-modin-project#1869: issue fix Signed-off-by: Alexander Myskov <alexander.myskov@intel.com> * FIX-modin-project#1869: remove comment Signed-off-by: Alexander Myskov <alexander.myskov@intel.com> * FIX-modin-project#1869: sort by _handle_level_agg Signed-off-by: Alexander Myskov <alexander.myskov@intel.com>
Signed-off-by: Alexander Myskov alexander.myskov@intel.com
What do these changes do?
flake8 modin
black --check modin
git commit -s