Conversation
There was a problem hiding this comment.
Not sure why you remove this tool, any reason?
There was a problem hiding this comment.
ooh but what is that ...
Maybe my branch was not up to date ...
There was a problem hiding this comment.
I finish improving here I'll open another PR on a up to data branch ...
| </param> | ||
| <when value="yes"> | ||
| <param name="operator" type="text" label="Enter operator name"/> | ||
| <param name="qc_convention" type="text" value="ARGO" label="Enter QC convention (default: ARGO)"/> |
There was a problem hiding this comment.
What else is possible, how do people now. Should this be a select?
There was a problem hiding this comment.
Yes probably ! i'll ask to imrpove that
| <option value="no">No, I don't have my a configuration file, I use the default one</option> | ||
| <option value="yes">Yes, I do have my own configuration file</option> | ||
| </param> | ||
| <when value="yes"> |
There was a problem hiding this comment.
Just saying most of the label and help text I do not understand. Is it expected that this tool is only used by ODV experts?
There was a problem hiding this comment.
Yes pretty much, We need to improve a bit the labels and the help but this not for beginners but for ODV experts
Co-authored-by: Björn Grüning <bjoern@gruenings.eu>
Co-authored-by: Björn Grüning <bjoern@gruenings.eu>
Co-authored-by: Björn Grüning <bjoern@gruenings.eu>
Co-authored-by: Björn Grüning <bjoern@gruenings.eu>
|
OPening up on a clean branch oups |
2 participants
Add new ocean biogeochemical tool.
TO DO