-
Notifications
You must be signed in to change notification settings - Fork 5
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
class error in wdpa_clean #19
Comments
Are you @jeffreyhanson around at the sri2020.org conference? Would be great meeting you. |
Hmm - that's strange. Could you please post a reproducible example showing this? Are you using No, I'm not attending that conference unfortunately - it looks pretty interesting! Yeah - it would be great to catch up. I'm based in Palmerston North in New Zealand - I don't suppose you're travelling to New Zealand while you're in the region? |
Well no I didnt convert the WDPA from (sorry it is in German, but the error remains the same) |
update: I now tried to clean SOM with the development version of |
its your go to close it in case you are good to go ;-) |
Great - glad to hear it's working. I'll close this issue now then. In the future, if you feel that you've solved an issue that you've opened, please feel free to close it yourself :) |
Strange... with a changed precision, the error comes up again. Would be great if you could have a look at it.
|
So this class error still is the bottleneck in my progress... |
Per #22, could you please post a complete reproducible example showing the issue you're experiencing? |
Hi – sorry for not replying yet. You can find the reproducible example above. However, it does not pop up always, I have no clue why. Did you change anything in the development version that could have lead to fixing the class error? |
Congrats to your paper, very nice reading! In regard of my work on the WPDA, using |
Thanks! It looks like the issue with Somalia is that all the protected areas are points and they have a REP_AREA of zero - meaning that there are no "spatially valid non-empty" geometries for this country. I'll add in a patch to return an empty result (i.e. sf object with zero rows) for such cases. |
Ok - I've just pushed a new version to GitHub that works for Somalia. Thanks for the reminder! |
After trying around with the
sp
package for quite a bit, I will now try to flatten the WDPA withwdpar
. The first error I came across, however, is as follows:It came in the WDPA subset for "SOM" (Somalia). Would be great if you can have a look at it.
Thanks and best regards,
Jonas
The text was updated successfully, but these errors were encountered: