eodms-sgdot / py-eodms-rapi

EODMS RAPI Client is a Python3 package used to access the REST API service provided by the Earth Observation Data Management System (EODMS) from Natural Resources Canada.
Other
13 stars 4 forks source link

Downloaded files do not use filetype specified by parameters #15

Open probinso opened 2 years ago

probinso commented 2 years ago

When I place an order and specify the packagingFormat as described in the documentation, the parameter is ignored. I Also never received an email for the order when overloading the response destination with NOTIFICATION_EMAIL_ADDRESS

...

>>> params = [
        {"packagingFormat": "TARGZ"},
        {"NOTIFICATION_EMAIL_ADDRESS": "philiprobinson@globalfishingwatch.org"}
]
>>> _ = api.order(orders, parameters=params)
>>> api.download(_, Path('output'))
| EODMSRAPI | Getting list of current orders...
| EODMSRAPI | Downloading image with Record Id 12392382 (RCM3_OK1670273_PK1761090_3_SC50MA_20210901_054854_HH_HV_GRD.zip).
| EODMSRAPI | RCM3_OK1670273_PK1761090_3_SC50MA_20210901_054854_HH_HV_GRD.zip: 100%|███████████████████| 1.28G
probinso commented 2 years ago

I was able to confirm that NOTIFICATION_EMAIL_ADDRESS does not overload the email address, by retrieving the order email from my account email.

probinso commented 2 years ago

I was informed that there might be a .SAFE standards compliance reason to return a .zip instead of a .tar.gz. That makes sense, and would mean that packagingFormat is only an issue with documentation.

It would still be nice to overload the notified email if that is possible.

probinso commented 2 years ago

Any news on this overloading of email bug? Is the backend code open source as well for this?

willmacs commented 2 years ago

@probinso, EODMS support for packagingFormat is being looked at by our vendor. It is planned to be fixed in an upcoming release of EODMS.