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

Value passing in an execute request. #385

Merged
merged 2 commits into from
Mar 4, 2024

Conversation

pvretano
Copy link
Contributor

Allow value passing to be specified per input rather than requiring thes erver to support both by-value and by-refernce in all cases. The default value for the valuePassing parameter is ["byValue","byReference"] to maintain backward compatability.

Closes #369

server to support both by-value and by-refernce in all cases.
@bpross-52n
Copy link
Contributor

SWG meeting from Dec 11th: maxItems will be removed from the enumeration. Enum will be made extensible.

@fmigneault
Copy link
Contributor

@pvretano
Sorry I couldn't assist this morning. Can you add precision about valuePassing?
Is it an extra parameter to be added for each input of each process description?
Can the server specify a conformance class it supports server-wide in the conformsTo response?

@jerstlouis
Copy link
Member

jerstlouis commented Dec 11, 2023

@fmigneault My understanding is that it is to be explicitly specified for each input of each process description

@pvretano
Copy link
Contributor Author

@fmigneault, @jerstlouis is correct. The valuePassing parameter can be specified for each input in a process description. If it is not specified the default value defined by the specification is ["byValue","byReference"] which is backward compatible with the current version of processes.

Question: Would it be of value to also add a valuePassing parameter at the top level of the process description to be a sort of global default? It would alleviate having to repeat the same values over and over and to also override the specification default.

@jerstlouis
Copy link
Member

@pvretano My opinion is that would not be very helpful, because the supported values are very likely to depend a lot more on the type of inputs.

e.g., a number or text input would normally be supported "byValue" and a GeoTIFF "byReference".

@fmigneault
Copy link
Contributor

I agree. I prefer to have them repeated in each input than having to cross-validate with the process description each time.

@bpross-52n
Copy link
Contributor

SWG Meeting from January 22nd: Remove min/max items and make the enum extensible.

@bpross-52n bpross-52n merged commit 1b28bb8 into opengeospatial:master Mar 4, 2024
1 check passed
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.

Execution endpoint input specification should be separate conformance class?
4 participants