Skip to content

Conversation

@itholic
Copy link
Contributor

@itholic itholic commented Mar 13, 2020

This PR proposes unique for MultiIndex and integration with existing Index.unique

>>> ks.MultiIndex.from_tuples([("A", "X"), ("A", "Y"), ("A", "X")]).unique()
MultiIndex([('A', 'X'),
            ('A', 'Y')],
           )

@codecov-io
Copy link

codecov-io commented Mar 13, 2020

Codecov Report

Merging #1342 into master will increase coverage by 0.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1342      +/-   ##
==========================================
+ Coverage   95.23%   95.25%   +0.01%     
==========================================
  Files          34       34              
  Lines        7538     7539       +1     
==========================================
+ Hits         7179     7181       +2     
+ Misses        359      358       -1
Impacted Files Coverage Δ
databricks/koalas/indexes.py 96.73% <100%> (+0.21%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b248a21...f6edcf0. Read the comment docs.

Copy link
Collaborator

@ueshin ueshin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@HyukjinKwon HyukjinKwon merged commit e4b114b into databricks:master Mar 15, 2020
@itholic itholic deleted the i_unique branch March 17, 2020 05:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants