Eliminate confusing job option and format option handling in `create_job` (and related)
@mbuchhorn pointed me to this snippet from doc: https://github.com/Open-EO/openeo-python-client/blob/7c321be49bcf6f1dbd091f5f4f4ea1c0163ed7e5/docs/cookbook/sampling.md?plain=1#L37-L42
I find this extremely confusing: the sample_by_feature looks like a job option this way, while the kwargs will be passed as format option:
https://github.com/Open-EO/openeo-python-client/blob/7c321be49bcf6f1dbd091f5f4f4ea1c0163ed7e5/openeo/rest/datacube.py#L2411-L2423
Note that there is a dedicated argument job_options for the job options, but the kwargs argument goes to format options, which is an optional feature here actually. I think we should get rid of this backwards argument handling
Same problem with:
-
DataCube.create_job(..., **format_options) -
DataCube.execute_batch(..., **format_options) -
VectorCube.create_job(..., **format_options) -
VectorCube.execute_batch(..., **format_options)
I had the same problem yesterday where I tried to pass a title to execute_batch but it treated it as a format option. I believe I got an error back so I changed it to create_job.
I had the same problem yesterday where I tried to pass a title to execute_batch but it treated it as a format option. I believe I got an error back so I changed it to create_job.
aargh same here: I was looking into CDSE integration tests that uses cube.execute_batch(title=... and couldn't understand why the tile was not showing up in the job registry
aargh same here: I was looking into CDSE integration tests that uses cube.execute_batch(title=... and couldn't understand why the tile was not showing up in the job registry
fixed with #652