Groupby.idxmax and Groupby.idxmin

[RainFung]

Now only support one-level index

[codecov-io]

Codecov Report

Merging #649 into master will decrease coverage by 2.06%.
The diff coverage is 95%.

@@            Coverage Diff             @@
##           master     #649      +/-   ##
==========================================
- Coverage   93.46%   91.39%   -2.07%     
==========================================
  Files          32       32              
  Lines        5324     5358      +34     
==========================================
- Hits         4976     4897      -79     
- Misses        348      461     +113

Impacted Files	Coverage Δ
databricks/koalas/missing/groupby.py	100% <ø> (ø)	⬆️
databricks/koalas/groupby.py	87.65% <95%> (+0.73%)	⬆️
databricks/koalas/usage_logging/__init__.py	23.14% <0%> (-74.08%)	⬇️
databricks/koalas/usage_logging/usage_logger.py	50% <0%> (-50%)	⬇️
databricks/koalas/__init__.py	74.35% <0%> (-10.26%)	⬇️
databricks/conftest.py	90.69% <0%> (-6.98%)	⬇️
databricks/koalas/frame.py	93.78% <0%> (-0.76%)	⬇️
databricks/koalas/internal.py	95.71% <0%> (-0.48%)	⬇️
databricks/koalas/plot.py	93.7% <0%> (-0.4%)	⬇️

---

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 ea61ad9...0918ea9. Read the comment docs.

[RainFung]

wait to be code reviewed.

[HyukjinKwon]

Looks fine otherwise

[RainFung]

I do nothing with groupby.rank why i failed in groupby.rank. Can you help me check it?

[softagram-bot]

Softagram Impact Report for pull/649 (head commit: c5af9a9)

⭐ Change Overview

(Open in Softagram Desktop for full details)

📄 Full report

• Permalink: Full report for pull/649

Give feedback on this report to [email protected]