-
Notifications
You must be signed in to change notification settings - Fork 38
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
Add definitions for expand_dimensions to the Makie extension #747
Conversation
Arrr, gather 'round, ye scurvy dogs, and listen well! The infamous AbstractMatrix band o' pirates, scourge o' the high seas, met their end not by the hand o' the navy, but by those meddlin' kids! Aye, it be true! Just as they were about to loot a treasure-laden galleon, them pesky young'uns swooped in with their clever tricks and caught the lot o' them red-handed. Now, the once-feared AbstractMatrix pirates be languishin' in the brig, their days o' plunderin' and pillagin' naught but a distant memory. Yarrr!
@@ -503,7 +510,7 @@ function _keywords2dimpairs(x, y) | |||
end | |||
end | |||
|
|||
_floatornan(A::AbstractArray{<:Union{Missing,<:Real}}) = _floatornan32.(A) | |||
_floatornan(A::AbstractArray{<:Union{Missing,<:Real}}) = _floatornan64.(A) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
64 is the default now? Nice
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep!
Looks like a Plots.jl version conflict causing some form of downgrade in the nightly CI |
Ah right. I've just been ignoring nightly 😅 |
Nice lol. If all the CI passes then this looks good to merge to me - I can confirm that my GeoMakie issue is resolved with this. |
Partially fix #720, fix the issue I was having with plotting Rasters in GeoMakie's meshimage as well.