Skip to content

OBSGESSION LAI#415

Open
JanssenBrm wants to merge 5 commits intomainfrom
obsgession_lai
Open

OBSGESSION LAI#415
JanssenBrm wants to merge 5 commits intomainfrom
obsgession_lai

Conversation

@JanssenBrm
Copy link
Copy Markdown
Contributor

No description provided.

@JanssenBrm JanssenBrm changed the title Obsgession LAI: initial setup of records Obsgession LAI Mar 18, 2026
@JanssenBrm JanssenBrm changed the title Obsgession LAI OBSGESSION LAI Mar 18, 2026
@algorithm-services-catalogue
Copy link
Copy Markdown

algorithm-services-catalogue bot commented Mar 18, 2026

🔍 Catalogue's Preview Site Deployed

Your changes have been deployed to the preview site:

🔗 Preview URL: https://esa-apex.github.io/apex-algorithms-catalogue-web/pr-preview/pr-415/

This preview will be updated automatically when you push new changes to your PR.

@JanssenBrm
Copy link
Copy Markdown
Contributor Author

@DeRooBert - I'm having an issue with updating UDP based on the provided description. I have installed the eo_processing package, but it is generating errors for me:

(apex_algorithms_qa) ➜  openeo_udp git:(obsgession_lai) ✗ python udp_generation.py 
Traceback (most recent call last):
  File "/Users/bramjanssen/projects/apex/apex_algorithms/algorithm_catalog/obsgession/udp_obsgession_w23_lai/openeo_udp/udp_generation.py", line 131, in <module>
    processing_options = get_advanced_options(
        provider=provider,
    ...<7 lines>...
        skip_check_S2=True,
    )
  File "/Users/bramjanssen/projects/apex/eo_processing/src/eo_processing/config/settings.py", line 377, in get_advanced_options
    raise ValueError(f'parameter for target_crs must be an integer value.')
ValueError: parameter for target_crs must be an integer value.

@DeRooBert
Copy link
Copy Markdown

@JanssenBrm it seems that a commit was not yet in dev. I asked for a pull request. After the pull request the problem should be solved. One extra remark is that it will be much easier to just adapt the json and not rerun it since there are plenty of manual stuff todo. Since the openeo has checks as well that do not go well with parameters.

@DeRooBert
Copy link
Copy Markdown

@JanssenBrm the mentioned pull request is in the dev branch of eo_processing

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.

2 participants