Skip to content

Conversation

@itholic
Copy link
Contributor

@itholic itholic commented Sep 3, 2019

It is very similar to #735 ,

Same problem that rpow and rfloordiv do not use proper operator in DataFrame.

스크린샷 2019-09-03 오후 5 47 17

  • In addition, Found such untested(not a few) operation functions,

스크린샷 2019-09-04 오전 12 37 28

so i've fixed them, could someone check this change maybe when available.

Thanks.

@codecov-io
Copy link

codecov-io commented Sep 3, 2019

Codecov Report

Merging #740 into master will increase coverage by 0.34%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #740      +/-   ##
==========================================
+ Coverage   94.35%   94.69%   +0.34%     
==========================================
  Files          32       32              
  Lines        5580     5580              
==========================================
+ Hits         5265     5284      +19     
+ Misses        315      296      -19
Impacted Files Coverage Δ
databricks/koalas/frame.py 96.12% <100%> (+1.07%) ⬆️
databricks/koalas/base.py 94.87% <0%> (+1.28%) ⬆️
databricks/conftest.py 97.67% <0%> (+2.32%) ⬆️
databricks/koalas/__init__.py 85% <0%> (+2.5%) ⬆️

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 f9c5088...6240de9. Read the comment docs.

@softagram-bot
Copy link

Softagram Impact Report for pull/740 (head commit: 6240de9)

⭐ Change Overview

Showing the changed files, dependency changes and the impact - click for full size
(Open in Softagram Desktop for full details)

📄 Full report

Impact Report explained. Give feedback on this report to [email protected]

@HyukjinKwon HyukjinKwon merged commit d3a1052 into databricks:master Sep 4, 2019
@itholic itholic deleted the add-test-frame branch September 6, 2019 01:40
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