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

mask(::Vector{<:SDMLayer}, polygon) dispatch fix #345

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

gottacatchenall
Copy link
Member

this really should've dispatched on vectors with elements that are any subtype of SDMLayer, not strictly SDMLayer

@gottacatchenall
Copy link
Member Author

seems the toaster @ucdavis hosts geodata on is once again in use

@gottacatchenall
Copy link
Member Author

sigh

image

@gottacatchenall
Copy link
Member Author

looks like @ucdavis turned the toaster off for the holidays and have finally restarted it. i think this should be good to merge @tpoisot

@gottacatchenall
Copy link
Member Author

just kidding it worked for SimpleSDMDatasets but not for the overall docs

@gottacatchenall
Copy link
Member Author

gottacatchenall commented Jan 2, 2025

I think we need a UC Davis spoofer that provides objects of the same type expected in downstream tests for the "occasional" scenario where their servers throw errors

@gottacatchenall
Copy link
Member Author

@tpoisot okay this is actually mergeable fr fr

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.

1 participant