johntruckenbrodt / pyroSAR

framework for large-scale SAR satellite data processing
MIT License
512 stars 112 forks source link

handling of GEE image catalogs #190

Open ashishgitbisht opened 2 years ago

ashishgitbisht commented 2 years ago
johntruckenbrodt commented 2 years ago

Hi @ashishgitbisht please provide more information than a question in the headline and the bug label.

ashishgitbisht commented 2 years ago

hello!! Query : I am working with imageCollection from Google earth engine, i am not downloading the datasets of sentinel 1. I am directly calling it using geemap API for earth engine.

I wanted to use pyroSAR, for pre-processing the dataset. so when i used 'identify' function from pyroSAR, it says object should be a string.

Thank you!!!!

On Tue, Feb 1, 2022 at 6:38 PM John Truckenbrodt @.***> wrote:

Hi @ashishgitbisht https://github.com/ashishgitbisht please provide more information than a question in the headline and the bug label.

— Reply to this email directly, view it on GitHub https://github.com/johntruckenbrodt/pyroSAR/issues/190#issuecomment-1026825348, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHF7PVQZWGFHDODPVN225RDUY7LOJANCNFSM5NIOL4LA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you were mentioned.Message ID: @.***>

pbrotoisworo commented 2 years ago

I'm curious, what are you trying to do with the data? Why can't you process it in Earth Engine? You need a specialized Earth Engine API like geemap or the official EE package to interact with Earth Engine data. It is designed for cloud computing. From what I know, you cannot use external tools (SNAP, ArcGIS, pyroSAR, etc) with Earth Engine. You can only use Earth Engine methods.

Earth Engine already preprocesses Sentinel-1 data.

EDIT: I saw you are using geemap already.

ashishgitbisht commented 2 years ago

Yes I am already using geemap to integrate Earth engine with Python in jupyter notebook. Also the earth engine data is good and pre processed but it doesn not qualify to be ARD (analysis ready data) as per CARD4L. I applied certain pre-processing on S1-GRD to convert it into ARD data, that is done in earth engine, but then i came across pyroSAR. So i was interested to be able to use pyroSAR for pre-processing dataset. Interesting fact is I dont have dataset stored in local directory, its rather a python object or image collection object which has dataset. How do we then pass into pyroSAR for further pre-processing to make it ARD data??? thats largely my query. thanks for prompt responses!!

pbrotoisworo commented 2 years ago

You cannot use pyroSAR with Earth Engine directly. One of the limitations of Earth Engine is that if you are using their cloud resources it only allows the methods in the official documentation. It will not allow external software to modify the data on their server.

https://developers.google.com/earth-engine/apidocs/ee-image image

johntruckenbrodt commented 2 years ago

Hi @ashishgitbisht and @pbrotoisworo. I am not so familiar with GEE to be honest. The last thing I know is that the data is processed but without the RTC correction (Small 2011). RTC correction has to be done in radar geometry so it is not possible with the already geocoded GEE data. There are however other methods to apply terrain correction to GEE-processed images, e.g. Vollrath et al. 2020. Andreas Vollrath is the main developer of OpenSarToolkit. Perhaps there you can find an implementation. I don't see how working with GEE image objects in pyroSAR would work. As @pbrotoisworo pointed out, this lets you apply GEE operations on the platform, whereas pyroSAR needs access to a local file system. It is a very interesting thought to extend pyroSAR to some GEE functionality. But this would require some amount of work to blend it in with the existing functionality. pyroSAR is currently not designed for such cloud workflows. @ashishgitbisht I think you might be better off developing your own prototype for now. Once you have a working prototype we could talk again and see how we can merge it.