Skip to content
This repository was archived by the owner on Aug 29, 2023. It is now read-only.

703 nf compute data frame #711

Merged
merged 4 commits into from
Jul 13, 2018
Merged

703 nf compute data frame #711

merged 4 commits into from
Jul 13, 2018

Conversation

forman
Copy link
Member

@forman forman commented Jul 13, 2018

This will close #703.

Copy link
Member

@JanisGailis JanisGailis left a comment

Choose a reason for hiding this comment

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

Looks quite neat!

@codecov-io
Copy link

codecov-io commented Jul 13, 2018

Codecov Report

Merging #711 into master will increase coverage by 0.2%.
The diff coverage is 95.08%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master     #711     +/-   ##
=========================================
+ Coverage   76.65%   76.86%   +0.2%     
=========================================
  Files          81       81             
  Lines       12043    12151    +108     
=========================================
+ Hits         9232     9340    +108     
  Misses       2811     2811
Impacted Files Coverage Δ
cate/ops/utility.py 84.41% <100%> (+0.63%) ⬆️
cate/ops/arithmetics.py 96.8% <94.73%> (-2.14%) ⬇️
cate/cli/main.py 64.47% <0%> (-0.13%) ⬇️
cate/ops/normalize.py 100% <0%> (ø) ⬆️
cate/util/monitor.py 93.4% <0%> (+0.5%) ⬆️
cate/core/opimpl.py 95.22% <0%> (+0.98%) ⬆️
cate/ops/io.py 84.66% <0%> (+1.01%) ⬆️

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 cfa9f47...bf05f15. Read the comment docs.

Copy link
Contributor

@hans-permana hans-permana left a comment

Choose a reason for hiding this comment

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

Looks good!

@forman forman merged commit 2c6c3ba into master Jul 13, 2018
@forman forman deleted the 703-nf-compute_data_frame branch September 10, 2018 16:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add operations compute_dataset() and compute_data_frame()
4 participants