-
Notifications
You must be signed in to change notification settings - Fork 26
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
Error in vignettes code: Unknown WKB type (648)! Full WKB type number was (50331648)
#154
Comments
This looks like https://github.com/r-spatial/spdep/blob/main/vignettes%2Fsids.Rmd#L51. Your platform appears to be rather outdated, so the underlying problem is not connected to this package. Can |
Please run and report all output:
If |
@rsbivand Thank you very much. Here is what I get:
Plot generation seemed to have worked fine. Should I open an issue with GEOS upstream then? |
Thanks! Could you try:
If that fails, next:
|
The last command launched X window, but an empty one. |
Error caused by missing
may work, the culprit seems to be
|
If I proceed after the first warning, then it the end nothing happens, no error, no output:
|
OK, returning to the original script:
run |
Running this, I got plot with numbers in the end:
And also from the second code. Result looks the same, perhaps, without comparing numbers at least. |
Original script version with traceback:
|
This reason for placing the number at the centroid of the largest polygon is that one county is made up of several parts. Thanks for the traceback! |
From https://github.com/r-spatial/lwgeom/blob/9d8078cbfcc33d2739344fa60b9188c6612fcd1d/R/geod.R#L13-L19 could you try:
|
This fails:
|
Sorry, should have set the coordinate reference system to match:
|
And this triggered the error we have seen initially!
|
OK, thanks, I'll raise an issue with |
Thank you! P. S. Aside of bitness and endianness there is one more potential source of obscure breakages – Darwin ppc ABI uses 4-byte bools. So if some alignments or size of structure assume 1-byte bool and that matters to the code execution, that may cause trouble. |
@barracuda156 Could you please check the vignette as just updated in 1fb4943 ? |
Does this persist if you set
? |
No, that was #154 (comment), second variant, vignette updated to that. lwgeom seems to fall over on big-endian WKB. |
Fixed upstream. |
I get an error in vignettes here:
Any idea what goes wrong?
The text was updated successfully, but these errors were encountered: