Skip to content

Conversation

@johnnychen94
Copy link
Member

Images v0.24.0 is somehow buggy for ColorVectorSpace 0.9 so we need to introduce patch #955 to fix it. This release resolves the compatibility issue #954 (comment) found by our Sharlock @kimikage.

This will be the last release that compatible to ImageCore 0.8.

@codecov
Copy link

codecov bot commented Apr 27, 2021

Codecov Report

Merging #956 (a834129) into master (21ddf3c) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #956   +/-   ##
=======================================
  Coverage   85.96%   85.96%           
=======================================
  Files           9        9           
  Lines        1026     1026           
=======================================
  Hits          882      882           
  Misses        144      144           
Impacted Files Coverage Δ
src/Images.jl 14.28% <ø> (ø)

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 21ddf3c...a834129. Read the comment docs.

@johnnychen94 johnnychen94 merged commit bff89d1 into master Apr 27, 2021
@johnnychen94 johnnychen94 deleted the jc/images002401 branch April 27, 2021 07:24
@kimikage
Copy link

Ah...Images.complement !== ColorVectorSpace.complement
Nevermind. It is not a practical problem.

(I was more preoccupied with my own mistake.:sob:)

@johnnychen94
Copy link
Member Author

johnnychen94 commented Apr 27, 2021

I'll live with that because Images doesn't reexport ColorVectorSpace.

However, ImageCore v0.9 will reexport ColorVectorSpace so I need to work out the deprecation before we can release the new version compatible with ImageCore 0.9.

@kimikage
Copy link

Yes, I mean...

julia> using Images

julia> complement(Gray(0.2)) # imported into `Main`
Gray{Float64}(0.8)

julia> using ColorVectorSpace
WARNING: using ColorVectorSpace.complement in module Main conflicts with an existing identifier.

julia> complement(Gray(0.2)) # still imported
Gray{Float64}(0.8)
julia> using Images

julia> using ColorVectorSpace

julia> complement(Gray(0.2))
WARNING: both ColorVectorSpace and Images export "complement"; uses of it in module Main must be qualified
ERROR: UndefVarError: complement not defined

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