Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Updates for new type system and tuple-printing #21

Merged
merged 3 commits into from
Feb 13, 2017
Merged

Updates for new type system and tuple-printing #21

merged 3 commits into from
Feb 13, 2017

Conversation

timholy
Copy link
Member

@timholy timholy commented Feb 11, 2017

This fixes several breakages on julia 0.6. Needs JuliaLang/julia#20568 and affected by JuliaLang/julia#20569. But at least this way we're better prepared.

@@ -4,6 +4,7 @@ module ImageCore

using Colors, FixedPointNumbers, MappedArrays, Graphics, ShowItLikeYouBuildIt
using OffsetArrays # for show.jl
using ColorTypes: colorant_string
Copy link

Choose a reason for hiding this comment

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

should be listed in REQUIRE if used directly

Copy link
Member Author

Choose a reason for hiding this comment

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

Indeed it should. Damn, you're impressive.

Copy link

Choose a reason for hiding this comment

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

observant, picky, OCD perhaps

this direct dependency stuff will hopefully be baked into the way Pkg3 environments work so it'll get auto-detected

@codecov-io
Copy link

codecov-io commented Feb 11, 2017

Codecov Report

Merging #21 into master will decrease coverage by -0.55%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master      #21      +/-   ##
==========================================
- Coverage   90.77%   90.22%   -0.55%     
==========================================
  Files           9        9              
  Lines         477      481       +4     
==========================================
+ Hits          433      434       +1     
- Misses         44       47       +3
Impacted Files Coverage Δ
src/ImageCore.jl 100% <ø> (ø)
src/deprecated.jl 77.35% <ø> (-1.49%)
src/stackedviews.jl 94.91% <100%> (ø)
src/convert_reinterpret.jl 89.74% <100%> (ø)
src/colorchannels.jl 90.25% <75%> (-0.53%)

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 8b01c7e...d26d49e. Read the comment docs.

@timholy
Copy link
Member Author

timholy commented Feb 11, 2017

Switched to a workaround for JuliaLang/julia#20568.

The `include_string` stuff is to work around an inference problem on 0.5

Still not inferred properly on 0.6.
@timholy
Copy link
Member Author

timholy commented Feb 13, 2017

This requires a new tag in OffsetArrays to pass tests, and at least some versions of 0.6 have an inference problem with the StackedViews tests (not our CI, apparently). But, this allows ImageCore to at least build on 0.6 and perform most of its duties, which should end the next main reason the whole Images suite is not passing on 0.6.

@timholy timholy merged commit b31e927 into master Feb 13, 2017
@timholy timholy deleted the teh/0.6 branch February 13, 2017 19:18
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.

3 participants