process schema issues

[soxofaan]

Some more process schema issues I found while comparing the VITO and EODC process listings for openEOPlatform/documentation#11

• count: context parameter has no "default":null
• count: condition parameter has no "default":null
• count: condition , when given as child process: does not define return schema, and context parameter has no "default":null
• array_element: index schema should have "minimum": 0 (https://github.com/Open-EO/openeo-processes/blob/60fb5e22579f34010de42af71c3d2d70f2dea906/array_element.json#L25)
• array_element: return_nodata should have default "false": https://github.com/Open-EO/openeo-processes/blob/60fb5e22579f34010de42af71c3d2d70f2dea906/array_element.json#L48
• save_result: options parameter has default "{}" instead of {}
• sar_backscatter: most parameters seem have different defaults than https://github.com/Open-EO/openeo-processes/blob/master/proposals/sar_backscatter.json
• merge_cubes: context parameter has no "default":null
• merge_cubes overlap_resolver: has no "default":null, no return schema and its context parameter has no "default":null
• mask: replacement has no default null
• filter_bands: bands and wavelenghts have default "[]" instead of []
• load_collection: temporal_extent minItems/maxItems are floats intead of int
• between: exclude_max has default true instead of false like https://github.com/Open-EO/openeo-processes/blob/60fb5e22579f34010de42af71c3d2d70f2dea906/between.json#L70
• ndvi: target_band has no "default":null
• run_udf: has somewhere a default "{}" instead of {}, missing some patterns like in https://github.com/Open-EO/openeo-processes/blob/60fb5e22579f34010de42af71c3d2d70f2dea906/run_udf.json#L39-L55
• eq/neq: delta has no "default":null
• sort/order: nodata has no "default":null
• if: reject has no "default":null
• array_apply/array_filter/reduce_dimension: context parameter has no "default":null, "callback" processes have no return schema and context parameter has no "default": null
• rename_labels: has a "[]" default instead of [] somewhere, also uses anyOf instead of easier construct like https://github.com/Open-EO/openeo-processes/blob/60fb5e22579f34010de42af71c3d2d70f2dea906/rename_labels.json#L29-L33

[m-mohr]

I guess the default value issues arise from

openeo-eodc-driver/services/processes/processes/schema.py

Line 149 in 822dbd3

default = fields.String(allow_none=True)

, which states that the default value is String although it can be of any data type.