-
Notifications
You must be signed in to change notification settings - Fork 42
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Drop support for pre-0.4 API? #72
Comments
None of the other clients (except the openEO Hub) support multiple versions at the moment. For now, I'd say users of the 0.3 API could simply use the latest version available that supports 0.3. (I don't think there are many, maybe only the openeo-result-validation-engine?) Of course that means there must be a release for it, which is available for all other clients except Python, I think. So action items could be:
Related #68 |
Releases of the python client do exist: so feel free to stop supporting 0.3, and start with cleaning code where needed. |
Aha, I thought there were none because I couldn't fine them on PyPI. |
- cross-check check and streamline test_rastercube unit tests in the process
- cross-check check and streamline test_rastercube unit tests in the process
- cross-check check and streamline test_rastercube unit tests in the process
and remove `download` from `ImageCollection` base class. Also drop pre-0.4 support from download (issue #72)
Could no longer find major pre 0.4.0 pieces of code, so considering this fixed. |
Open question: is it necessary to keep supporting 0.3-style API in the python client? I have not really an idea whether there are actual users that use the python client with 0.3-only backends.
There are quite some differences between the 0.3 and 0.4 versions of the API (how process graphs work, terminology, etc), and supporting both in the python client code makes things a bit tedious and hard to maintain.
It would benefit speed of development and code quality if we could get rid of all the 0.3-related code and implementation details.
The text was updated successfully, but these errors were encountered: