search

openEOPlatform / openeo-classification

openEO based classification workflows & utilities, for landcover and crop mapping usecases.
Apache License 2.0
4 stars 1 forks source link

Only use `openeo_classification.resources` package for input resources #2

Open soxofaan opened 2 years ago

soxofaan commented 2 years ago

While working on #1 I noticed that openeo_classification/resources is mainly used for generated/output files.

I think this should be avoided and openeo_classification.resources should be solely for input resources

soxofaan commented 2 years ago

examples:

referenced resource doesn't even exist in git repo: https://github.com/openEOPlatform/openeo-classification/blob/494fabaefed54826395c7a3aa7f620456e7fa29d/src/openeo_classification/classify.py#L13

referenced resource doesn't even exist in git repo: https://github.com/openEOPlatform/openeo-classification/blob/494fabaefed54826395c7a3aa7f620456e7fa29d/src/openeo_classification/job_management.py#L83

https://github.com/openEOPlatform/openeo-classification/blob/494fabaefed54826395c7a3aa7f620456e7fa29d/src/openeo_classification/main.py#L87-L91

https://github.com/openEOPlatform/openeo-classification/blob/494fabaefed54826395c7a3aa7f620456e7fa29d/src/openeo_classification/prepare_data.py#L13

https://github.com/openEOPlatform/openeo-classification/blob/494fabaefed54826395c7a3aa7f620456e7fa29d/src/openeo_classification/prepare_data.py#L148

https://github.com/openEOPlatform/openeo-classification/blob/494fabaefed54826395c7a3aa7f620456e7fa29d/src/openeo_classification/sample.py#L53

https://github.com/openEOPlatform/openeo-classification/blob/494fabaefed54826395c7a3aa7f620456e7fa29d/src/openeo_classification/sample.py#L156