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.

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

added exceptions for ndvi process #290

Closed
wants to merge 1 commit into from

Conversation

bossie
Copy link

@bossie bossie commented Jun 9, 2020

They match the exceptions specified in the ndvi process description in openeo-processes.

@m-mohr
Copy link
Member

m-mohr commented Jun 9, 2020

Thanks for opening the PR, but the error list in the API is not meant to contain process specific errors. They are only defined in the processes itself to avoid dependencies between processes and the API (they are meant to be released independently).

@m-mohr m-mohr closed this Jun 9, 2020
@m-mohr
Copy link
Member

m-mohr commented Jun 9, 2020

We removed everything process specific for the next release in PR #275 as there were some left-overs...

@bossie bossie deleted the 60_ndvi_rewrite branch June 9, 2020 12:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants