Skip to content

Conversation

@Pratichhya
Copy link
Contributor

We noticed that a file was missing, likely during the migration process.
Nevertheless, it was already referenced in the process graph, so it does not change the graph.

@Pratichhya Pratichhya requested a review from HansVRP November 12, 2025 09:04
@algorithm-services-catalogue
Copy link

algorithm-services-catalogue bot commented Nov 12, 2025

🧹 Catalogue's Preview Site Cleanup

The preview site for this pull request has been cleaned up.

If you need a new preview, please reopen the PR or create a new one.

Copy link
Contributor

@HansVRP HansVRP left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We do not necessarily need to add this file. APEX per definition does not have to host the creation file

temporal_extent=date,
bands=bands)
s1_grd = s1_grd.sar_backscatter(coefficient='sigma0-ellipsoid')
s1_grd = s1_grd.rename_labels(dimension="bands", target=bands)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why is this rename_labels required?

spatial_extent=polygon,
temporal_extent=date,
bands=["SCL"])
mask = mask = scl.process("to_scl_dilation_mask", data=scl)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

isn't this the old method which got deprecated?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also mask = mask = ...

"""
base_biopar = process(
process_id="BIOPAR",
namespace="https://openeo.dataspace.copernicus.eu/openeo/processes/u:3e24e251-2e9a-438f-90a9-d4500e576574/BIOPAR",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we don't have an alternative for this?

:return:
"""
collections = None
for option in [
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I do not think its good practice to have these optional patterns in UDP generation. It defeats the purpose of being deterministic

@Pratichhya Pratichhya closed this Nov 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants