More defaults (#68)

* some defaults should not be set using 'default'

To implement the slightly complicated default value rules for
ResourceRequirement

* InputBinding.position default is 0.

* File.streamable default is false.

Author: mr-c

CommandLineTool.yml

@@ -180,10 +180,12 @@ $graph:
   fields:
     - name: position
       type: [ "null", int, Expression ]
+      default: 0
       doc: |
-        The sorting key.  Default position is 0. If the inputBinding is
-        associated with an input parameter, then the value of `self` in the
-        expression will be the value of the input parameter.  Input parameter
+        The sorting key.  Default position is 0. If a [CWL Parameter Reference](#Parameter_references)
+        or [CWL Expression](#Expressions_(Optional)) is used and if the
+        inputBinding is associated with an input parameter, then the value of
+        `self` will be the value of the input parameter.  Input parameter
         defaults (as specified by the `InputParameter.default` field) must be
         applied before evaluating the expression. Expressions must return a
         single value of type int or a null.
@@ -1165,6 +1167,10 @@ $graph:
         "_type": "@vocab"
     - name: coresMin
       type: ["null", long, float, Expression]
+      # default: 1
+      # Don't set the default here, we need to be able to distinguish between
+      # a value not provided and the textual default to be able to implement
+      # the rules written in the ResourceRequirement doc above
       doc: |
         Minimum reserved number of CPU cores (default is 1).
 
@@ -1201,6 +1207,12 @@ $graph:
 
     - name: ramMin
       type: ["null", long, float, Expression]
+      # default: 256
+      # Don't set the default here, we need to be able to distinguish between
+      # a value not provided and the textual default to be able to implement
+      # the rules written in the ResourceRequirement doc above.
+      # Consumers of ResourceRequirement must apply the default value algorithm
+      # themselves and not rely on schema-salad rules
       doc: |
         Minimum reserved RAM in mebibytes (2**20) (default is 256)
 
@@ -1219,6 +1231,12 @@ $graph:
 
     - name: tmpdirMin
       type: ["null", long, float, Expression]
+      # default: 1024
+      # Don't set the default here, we need to be able to distinguish between
+      # a value not provided and the textual default to be able to implement
+      # the rules written in the ResourceRequirement doc above
+      # Consumers of ResourceRequirement must apply the default value algorithm
+      # themselves and not rely on schema-salad rules
       doc: |
         Minimum reserved filesystem based storage for the designated temporary directory, in mebibytes (2**20) (default is 1024)
 
@@ -1237,6 +1255,12 @@ $graph:
 
     - name: outdirMin
       type: ["null", long, float, Expression]
+      # default: 1024
+      # Don't set the default here, we need to be able to distinguish between
+      # a value not provided and the textual default to be able to implement
+      # the rules written in the ResourceRequirement doc above
+      # Consumers of ResourceRequirement must apply the default value algorithm
+      # themselves and not rely on schema-salad rules
       doc: |
         Minimum reserved filesystem based storage for the designated output directory, in mebibytes (2**20) (default is 1024)
 

Process.yml

@@ -558,6 +558,7 @@ $graph:
 
     streamable:
       type: boolean?
+      default: false
       doc: |
         Only valid when `type: File` or is an array of `items: File`.