-
-
Notifications
You must be signed in to change notification settings - Fork 199
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
Define the concept of an "opaque string" #914
Comments
My understanding is that we meant
@tetron can provide more context |
Yes, that's basically it. The important thing is that workflow/tool expressions shouldn't try to parse it. |
The purpose being to allow the runner to swap in a value late in the evaluation process, after the workflow/tool expressions have had a chance to see it. |
Today-I-Learned about opaque data types, and opaque string. I had a look at the Wikipedia article, and also at the text where we use “opaque”.
This is easy to understand while reading the rest of the text. I'll try to incorporate something similar in a PR to update where “opaque” is used as I think that's clearer (saves the reader unfamiliar with the term to have to open a link or read a footer note.) |
I spent some time reading more about opaque strings, and it appears to be used in specs, some programming languages, and I found this Microsoft 365 doc, this AWS doc, and this NASA doc on DOI that use the term as well. Maybe we should leave “opaque” and “opaque string”, but clarify it somewhere else in the document? What others think? If not, maybe a replacement term could be placeholder string, always followed by a note that implementees must not rely on this value… |
We could insert a glossary after https://www.commonwl.org/v1.2/CommandLineTool.html#Terminology and define the terms there; they can be linked back from where they are used |
https://en.wikipedia.org/wiki/Opaque_data_type
Found in
https://www.commonwl.org/v1.2/CommandLineTool.html#Parameter_references
https://www.commonwl.org/v1.2/CommandLineTool.html#Runtime_environment
The text was updated successfully, but these errors were encountered: