-
Notifications
You must be signed in to change notification settings - Fork 12
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
Strange libgdal.so.28 issue #120
Comments
@jlgutenson cool work. looks great. do you know that I publish a go-consequences container that you can call like you want? I also notice that you build on the gdal image, and that you do not grab build essential. In my prod container i have the following lines:
I can understand not wanting build essential in the prod container, but that is a potential for where this might differ. I see that you have your hazard provider as a shapefile - did you update the code locally to work with that hazard provider type? |
Hey @HenryGeorgist, good to hear from you! Thanks for the quick response! I didn't know you had your image published. That sounds like exactly what I need. Is it this guy? That shapefile was actually supposed to be tiff file/depth grid reference. Good eye! If so, I tried running However, I'm not finding the |
hmm. that looks ok. did you try v0.0.1? |
i can reproduce it on version 0.0.2, looks like the exported command was /main instead of ./main |
reproduced with my checked in prod container. |
Good suggestion! I can confirm that your v0.0.1 image is running. But my outputs (I tried json, geopackage, and shapefile) with v0.0.1 are not complete once the analysis ends. That might be a connection issue on my end. |
Hey @HenryGeorgist, after further analysis, I'm having some issues with the v0.0.1 image's outputs that I don't believe are memory or internet connection related. I had ChatGPT analyze the file and it wasn't able to clean the JSON to make it usable in a GIS. Here is an attached JSON output for reference. |
on first glance it is missing a closing square brace at the end ] of the features array, and then it would need to close the feature collection object with a curly brace } the end should look essentially like this }}]} it currently looks like this }}, this should happen in the "Close" methodology on the results writer. it is possible that the close method wasnt defered in version 0.0.1 for the geojson results writer. i typically use geopackage. but if the close method is not deferred on it too then we would have a problem of not flushing the last transaction. i think geojson and editing the last two characters may be enough.... chat gpt? seriously? |
I gave }}]} at the end a whirl. No luck, unfortunately. With v0.0.1, the Geopackage option keeps giving me only one damaged building. The one building is in the depth grid and pulling the correct depth but only writing the one location for some reason. I tried the shapefile option as well and it was completely empty. With ChatGPT, don't underestimate my laziness. I have it do the hard work and I just fill in where I need to or, in this case, I was stumped and I thought I'd see if it saw something in the JSON that I was missing. It seemed very much confused about the JSON issue. |
i sent you a file that worked for me in qgis after my edits. i use jsonlint.com to validate the json and ensure it meets the specification, it helps me point out the problems that need to be resolved. your description of geopackage seems consistent with the geojson writer in that it is not triggering the close method when the program is shutting down. looks like i need to fix the code one way or another eventually. |
Thank you! My email changed. Did you use [email protected] or [email protected]? We just invested in the paid version of ChatGPT. We're somewhat hooked. It can be really good for working from home without a rubber duck handy! But it also can lead you on some wild goose chases! I moved back to the original gdal issue that's plaguing v0.0.2. Still haven't figured that one out either. I'm not sure what GDAL version 2.8 dependencies there are in go-consequences. It may be worth mentioning that this is all in an effort to demonstrate how go-consequences can be used to forecast flood consequences, if provided a forecasted flood depth grid. |
i run the latest in a different container and this is what i use as my production container:
it uses the osgeo ubuntu full - which is not small and maybe is overkill. |
gah, but thats the same one you were using. which doesnt make sense |
I think I got the gdal issues in v0.0.2 solved (with the help of our friend ChatGPT :-))! Maybe you can confirm. I've attached a geotiff for easy access. Here is the modified Dockerfile.prod. It's just tricky the image into thinking it has a libgdal.so.28 file with a symbolic link.
I then built the Docker image: Then I can use the geojson to run go-consequences and output a geopackage:
|
that looks good to me if the results make sense and it runs! |
Sounds good, we will test a few different times and see if things make sense. Things seem correct for my Asheville test case. |
Describe the bug
I'm having an issue with a libgdal.so.28 dependency somewhere in go-consequences when setting up and running go-consequences using Docker Desktop on Windows. The error is:
/app/main: error while loading shared libraries: libgdal.so.28: cannot open shared object file: No such file or directory
after attempting to run a flood consequence assessment with the command:
docker run --mount type=bind,source="C:\Users\jlgut\OneDrive\Desktop\go-consequences",target=/data go-consequences-jlg:latest ./main "/data/joseph_test.json"
To Reproduce
Steps to reproduce the behavior:
docker build -f Dockerfile.prod -t go-consequences-jlg .
docker run --mount type=bind,source="C:\Users\jlgut\OneDrive\Desktop\go-consequences",target=/data go-consequences-jlg:latest ./main "/data/joseph_test.json"
/app/main: error while loading shared libraries: libgdal.so.28: cannot open shared object file: No such file or directory
Expected behavior
I was expecting to have go-consequences run this test-case (i.e., joseph_test.json):
The text was updated successfully, but these errors were encountered: