search

imedslab / solt

Streaming over lightweight data transformations
https://oulu-imeds.github.io/solt/
MIT License
264 stars 19 forks source link

Add IntensityRemap transform #45

Closed soupault closed 4 years ago

soupault commented 4 years ago

Hi, If this is of any interest, I will start to work on the tests.

lext commented 4 years ago

@soupault go ahead!

lext commented 4 years ago

Codecov Report

Merging #45 into master will not change coverage by %. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##            master       #45   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            8         8           
  Lines         1297      1319   +22     
=========================================
+ Hits          1297      1319   +22

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 2e8aa3d...4a0f50e. Read the comment docs.

codecov-io commented 4 years ago

Codecov Report

Merging #45 into master will not change coverage. The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #45   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           8      8           
  Lines        1297   1322   +25     
=====================================
+ Hits         1297   1322   +25
Impacted Files Coverage Δ
solt/transforms/__init__.py 100% <ø> (ø) :arrow_up:
solt/transforms/_transforms.py 100% <100%> (ø) :arrow_up:
solt/core/_core.py 100% <0%> (ø) :arrow_up:

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 2e8aa3d...5c41bea. Read the comment docs.

soupault commented 4 years ago

Ready for review.

soupault commented 4 years ago

That's all I can think about regarding the testing.