From 2667733563e9eeac00879b0d316d858b4c2dcb5a Mon Sep 17 00:00:00 2001 From: Stefaan Lippens Date: Mon, 14 Oct 2024 17:15:29 +0200 Subject: [PATCH] Issue #604/#644 add some todo note for furtherdevelopment ideas --- openeo/extra/job_management.py | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/openeo/extra/job_management.py b/openeo/extra/job_management.py index b22ff41db..1125f7e2e 100644 --- a/openeo/extra/job_management.py +++ b/openeo/extra/job_management.py @@ -1006,6 +1006,9 @@ class UDPJobFactory: .. versionadded:: 0.33.0 """ + # TODO: find a better class name (e.g. eliminate over-specificity of "UDP", + # or avoid "factory" as technical mumbo-jumbo)? + def __init__( self, *, @@ -1048,6 +1051,8 @@ def start_job(self, row: pd.Series, connection: Connection, **_) -> BatchJob: :param row: The row in the pandas dataframe that stores the jobs state and other tracked data. :param connection: The connection to the backend. """ + # TODO: refactor out some methods, for better reuse and decoupling: + # `get_arguments()` (to build the arguments dictionary), `get_cube()` (to create the cube), process_definition = self._get_process_definition(connection=connection) process_id = process_definition.id