tsdataclinic / PMLDataPipeline

A pipeline for detecting macro-plastics in coastal waters using Sentinel 2 data
Apache License 2.0
19 stars 13 forks source link

Error in download_features.py #6

Open Dwat2505 opened 2 years ago

Dwat2505 commented 2 years ago

I'm interested in using this project in my thesis. I am willing to run this before making any modifications to the same. However, I've been running into quite a few issues.

While running python download_features.py --features data/features.csv

I get the following error ImportError: cannot import name 'S2L1CWCSInput' from 'eolearn.io'

Also. I get the unresolved variables error before even running the files on this line https://github.com/tsdataclinic/PMLDataPipeline/blob/b6678e06d25fee2aec7a28982997578c497b98cf/download_features.py#L72

The variable bounds, bbox_list aren't defined in the above line

https://github.com/tsdataclinic/PMLDataPipeline/blob/b6678e06d25fee2aec7a28982997578c497b98cf/download_features.py#L78-L81

The variable USE_BANDS, band_names aren't defined in the above line.

@stuartlynn if you could help me set this up, it'd be very helpful for my thesis.

PierreTsr commented 2 years ago

I'm trying to make this work too for a project, but it seems that the repo is based on an old version of eo-learn.

When running, python download_scene.py --scene=scenes/Gulf.json I first had an import error: ImportError: cannot import name 'get_s2_pixel_cloud_detector' from 'eolearn.mask' Using eo-learn's documentation I tried to use the new syntax for cloud masking, but now I'm faced with the same error as you.

I tried with both the last version of eo-learn, and the one given in Pipfile.lock, 0.7.4. But none of them work. I have absolutely no experience with that package, and I had no luck trying to find the correct version by looking for the missing function in the release logs...

I think the easiest fix would be to get from @stuartlynn the correct version to use for eo-learn