MakieOrg / GeoMakie.jl

Geographical plotting utilities for Makie.jl
https://geo.makie.org
MIT License
161 stars 23 forks source link

GeoMakie.jl warnings during tests leading to precompilation failure #217

Closed ronisbr closed 1 month ago

ronisbr commented 2 months ago

Hi!

When precompiling GeoMakie.jl in tests, I am seeing this kind of warnings:

WARNING: Method definition isfinite(Union{GeometryBasics.AbstractPoint{Dim, T} where T where Dim, GeometryBasics.Vec{S, T} where T where S}) in module GeometryBasics at /home/runner/.julia/packages/GeometryBasics/ebXl0/src/fixed_arrays.jl:134 overwritten in module GeoMakie at /home/runner/.julia/packages/GeoMakie/vcMuZ/src/utils.jl:128.
ERROR: Method overwriting is not permitted during Module precompilation. Use `__precompile__(false)` to opt-out of precompilation.
WARNING: Method definition isfinite(Union{GeometryBasics.AbstractPoint{Dim, T} where T where Dim, GeometryBasics.Vec{S, T} where T where S}) in module GeometryBasics at /home/runner/.julia/packages/GeometryBasics/ebXl0/src/fixed_arrays.jl:134 overwritten in module GeoMakie at /home/runner/.julia/packages/GeoMakie/vcMuZ/src/utils.jl:128.
┌ Warning: Module GeoMakie with build ID ffffffff-ffff-ffff-0000-005c2cd9cc5a is missing from the cache.
│ This may mean GeoMakie [db073c08-6b98-4ee5-b6a4-5efafb3259c6] does not support precompilation but is imported by a module that does.
└ @ Base loading.jl:1948

(See here https://github.com/JuliaSpace/SatelliteAnalysis.jl/actions/runs/8863099460/job/24336758108#step:6:715)

Is this something know or a new issue?

asinghvi17 commented 2 months ago

Ah, that's the result of a new GeometryBasics release. I'll release a patch version with those removed shortly.

ronisbr commented 1 month ago

Hi @asinghvi17 !

I saw you bump the version but it was not registered. Is there something I can do to help?

asinghvi17 commented 1 month ago

Thanks for reminding me :D - release is in progress now.