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
In the geocode function RemoveThermalNoise step failed with a null pointer exception when a shapefile is specified #113
Comments
Hi @antoine-collet, |
Hello @johntruckenbrodt, |
Thank you very much for reporting this @antoine-collet, I was running crazy with this error. Will wait for the issue fix. However, I am surprised that there is not proper versioning on the .sh installers, so that we can easily rollback to previous versions. I have one version in local installed few months ago that works flawlessly, but a version installed just yesterday from this url http://step.esa.int/downloads/7.0/installers/esa-snap_sentinel_unix_7_0.sh is causing this error. I am running pipelines in a Docker without GUI access, hence the problem. |
Yep, that's been bothering me for a long time. Instead of new features I would love to see some better software packaging. There is not even a proper way to automatically update from the command line, only a workaround written by a former colleague of mine described here. |
Yeah, thanks for the link @johntruckenbrodt, and again, your library is just amazing! Super glad to be working with it! I am working on a fix to subset using external tools (rasterio) so that I can bypass it, let me know if it is of your interest to add it as a workaround. Apparently a similar issue with snap subset happened back in 2018, so I just want to cover my back. |
Hi @vfp1 really great you enjoy working with pyroSAR, thanks mate! It sure would be convenient to be able to subset S1 scenes in Python. I wonder though whether it's that easy. Subsetting the tif files should be relatively straight forward but all the metadata files like calibration and noise vectors might have to be adjusted as well. Seems like this is the file containing the relevant SNAP code. |
Thanks! Will keep you posted with this. |
Dear @johntruckenbrodt, is it possible to add more nodes to pyroSAR? I have been trying this for a while, I think one solution will be to add the Land-Sea Mask with vector masking capabilities. |
Hi @vfp1, do you mean adding nodes to pyroSAR in general or to the |
I am still having this issue.Is there any workaround for this? |
Wow! Such a long discussion and no fix in sight. Thanks a lot @clausmichele for providing this embarrassingly easy solution!!! I'll merge the PR in a minute. |
Hello,
I am having trouble geocoding scenes when specifying a shapefile in order to reduce processing time. There seems to be a problem with the RemoveThermalNoise step (which failed with an exception). It is to note the function is running smoothly if I do not specify the site shapefile.
which operating system are you using?
Ubuntu 18.4
which environment is pyroSAR running in?
A dockerised environement
which version of pyroSAR are you using?
Installed via pip: pyroSAR==0.11
which function of pyroSAR did you call with which parameters?
geocode(infile=scene, outdir='./experiment_v1/subset', tr=10, scaling='dB', shapefile='shape.shp')
if applicable, which version of SNAP or GAMMA are you using in pyroSAR?
SNAP version 7.0.0
the full error message
The text was updated successfully, but these errors were encountered: