allow using udp parameter for spatial/temporal extents

[jdries]

pull request to allow passing udp Parameter objects

[jdries]

@HansVRP @VincentVerelst in the frame of adding worldcereal UDP's to apex catalog, can someone take over this PR and check what is still relevant and needed today?
It's not super urgent, but we probably want to move this forward.

Related to: ESA-APEx/apex_algorithms#28

[HansVRP]

Main issue is that the linting is failing. the repo requires the use of pre-commit, black and isort

[HansVRP]

for sake of readibility, would it make sense to also introdue temporal_is_param = isinstance(temporal_extent, Parameter)

[HansVRP]

reading deeper into this _load collection I think it would be useful to modularize the getting the temporal and spatial extent.

I think something like:

 # Helper function for temporal extent processing
    def process_temporal_extent(temporal_extent):
        if isinstance(temporal_extent, Parameter) or temporal_extent is None:
            return temporal_extent
        return [temporal_extent.start_date, temporal_extent.end_date]

    # Helper function for spatial extent processing
    def process_spatial_extent(spatial_extent, fetch_type):
        if fetch_type == FetchType.TILE:
            if not isinstance(spatial_extent, BoundingBoxExtent) and not isinstance(spatial_extent, Parameter):
                raise ValueError("Please provide only a bounding box for tile-based fetching.")
            return spatial_extent if isinstance(spatial_extent, Parameter) else dict(spatial_extent)

        elif fetch_type == FetchType.POINT:
            if not isinstance(spatial_extent, GeoJSON) or spatial_extent.get("type") != "FeatureCollection":
                raise ValueError("Please provide a valid GeoJSON FeatureCollection for point-based fetching.")
            return spatial_extent

        elif fetch_type == FetchType.POLYGON:
            if isinstance(spatial_extent, GeoJSON):
                if spatial.get("type") != "FeatureCollection":
                    raise ValueError("Please provide a valid FeatureCollection type of GeoJSON.")
                return spatial_extent
            elif isinstance(spatial_extent, str):
                if not (spatial_extent.startswith("https://") or spatial_extent.startswith("http://")):
                    raise ValueError("Please provide a valid URL to a GeoParquet or GeoJSON file.")
                return connection.load_url(spatial_extent, format="Parquet" if ".parquet" in spatial else "GeoJSON")
            else:
                raise ValueError("Please provide a valid URL to a GeoParquet or GeoJSON file.")
        
        return spatial_extent

should work.

The only part which would not be able to be modularized is:

    if fetch_type == FetchType.POLYGON and isinstance(spatial_extent, (str, GeoJSON)):
        cube = cube.filter_spatial(spatial_extent)

thoughts?