Skip to content
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.

Sign up for GitHub

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

Jump to bottom

run_udf exceptions #273

Closed
soxofaan opened this issue Aug 17, 2021 · 1 comment
Closed

run_udf exceptions #273

soxofaan opened this issue Aug 17, 2021 · 1 comment
Milestone
1.2.0

Comments

@soxofaan
Copy link
Member

openeo-processes/run_udf.json

Lines 93 to 97 in 3dc5e78

"exceptions": {
"InvalidVersion": {
"message": "The specified UDF runtime version is not supported."
}
},

Why only an exception for invalid version, but not for invalid runtime?

Possible solutions:

  • add InvalidRuntime
  • rename to InvalidRuntime to cover both invalid versions and runtimes
  • cover everything with standard ProcessParameterInvalid
m-mohr added a commit that referenced this issue Aug 17, 2021
@m-mohr
Add InvalidRuntime to run_udf #273, fix changelog
203c1b6
@m-mohr
Copy link
Member

m-mohr commented Aug 17, 2021

Adding InvalidRuntime seems like a simple solution. See 203c1b6

@m-mohr m-mohr closed this as completed Aug 17, 2021
@m-mohr m-mohr added this to the 1.2.0 milestone Aug 17, 2021
soxofaan added a commit to Open-EO/openeo-python-driver that referenced this issue Aug 18, 2021
@soxofaan
Issue #84 use proper errors for invalid UDF Runtime or version
6984fe0 
also see Open-EO/openeo-processes#273
@m-mohr m-mohr mentioned this issue Dec 1, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
1.2.0
Development

No branches or pull requests
2 participants
@soxofaan @m-mohr