Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Support nesting for timeaxis over MappedArrays #16

Merged
merged 1 commit into from
Mar 21, 2017
Merged

Conversation

timholy
Copy link
Member

@timholy timholy commented Mar 21, 2017

Plus add tests of nesting behavior. This is the follow-up to JuliaImages/ImageCore.jl#30, and tests that the nesting behavior produces more obviously-useful results.

@codecov-io
Copy link

codecov-io commented Mar 21, 2017

Codecov Report

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

@@          Coverage Diff          @@
##           master    #16   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           2      2           
  Lines          79     80    +1     
=====================================
+ Hits           79     80    +1
Impacted Files Coverage Δ
src/ImageAxes.jl 100% <100%> (ø) ⬆️

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 8bd2356...acfe252. Read the comment docs.

@@ -3,7 +3,7 @@ __precompile__()
module ImageAxes

using Base: @pure, tail
using Reexport, Colors, SimpleTraits
using Reexport, Colors, SimpleTraits, MappedArrays
Copy link
Member

Choose a reason for hiding this comment

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

Needs an entry in REQUIRE

Also add tests of nesting behavior
@timholy timholy force-pushed the teh/traits_nesting branch from a78751b to acfe252 Compare March 21, 2017 10:44
@timholy timholy merged commit ff917a1 into master Mar 21, 2017
@timholy timholy deleted the teh/traits_nesting branch March 21, 2017 13:10
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants