Skip to content
This repository has been archived by the owner on Mar 6, 2023. It is now read-only.

Allow file format to be upper/lowercase in save_results #22

Closed
lforesta opened this issue Apr 7, 2021 · 4 comments
Closed

Allow file format to be upper/lowercase in save_results #22

lforesta opened this issue Apr 7, 2021 · 4 comments
Assignees

Comments

@lforesta
Copy link
Contributor

lforesta commented Apr 7, 2021

According to the openEO API spec: "Format names are allowed to be case insensitive throughout the API."

@lforesta lforesta self-assigned this Apr 7, 2021
@soxofaan
Copy link
Member

soxofaan commented Apr 7, 2021

This actually raises the question why it states "are allowed to be case insensitive".
Does this mean that a backend can choose to be case insensitive about format names?
Wouldn't it be better (for portability) to enforce that format names are case insensitive?
(same for Service names and Billing plans by the way)

cc @m-mohr

@m-mohr
Copy link
Member

m-mohr commented Apr 7, 2021

A back-end must accept formats case-insensitive to make it easier for users to use. We had a number of workshops where people had issues with that, e.g. the back-end exposed "GTiff" and people just typed "gtiff" in the client. This is purely for a better user experience. Maybe we need to improve wording in the API, this is clearly written from a client perspective.

@m-mohr
Copy link
Member

m-mohr commented Apr 7, 2021

See PR Open-EO/openeo-api#372

@clausmichele
Copy link
Member

This issue has been solved here #25

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants