Skip to content

sen4cap: processor integration#321

Merged
JanssenBrm merged 14 commits intomainfrom
sen4cap_processors
Feb 10, 2026
Merged

sen4cap: processor integration#321
JanssenBrm merged 14 commits intomainfrom
sen4cap_processors

Conversation

@JanssenBrm
Copy link
Contributor

No description provided.

@algorithm-services-catalogue
Copy link

algorithm-services-catalogue bot commented Jan 27, 2026

🧹 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.

@JanssenBrm
Copy link
Contributor Author

JanssenBrm commented Jan 27, 2026

Did the following minor fixes to the records:

  • Updated the keywords to match the predefined list of keywords
  • Fixed some small typos in the description
  • Removed the view-source: prefix from the TAO platform logo.

The following fixes should still be applied before merging:

image

@lnicola
Copy link
Contributor

lnicola commented Jan 28, 2026

Hi @JanssenBrm, for https://www.esa-sen4cap.org/ I see a valid certificate (valid from 24 Jan to 24 Apr), can you say what's wrong?

I can confirm the other issues, we'll notify you when fixed.

@JanssenBrm
Copy link
Contributor Author

@lnicola - The certificate issue with sen4cap.org is indeed fixed now. Yesterday it was showing me as expired (maybe it was due to caching).

@JanssenBrm JanssenBrm self-assigned this Jan 28, 2026
@JanssenBrm
Copy link
Contributor Author

Tagging @cosminudroiu for follow-up

@JanssenBrm JanssenBrm removed their assignment Jan 28, 2026
@lnicola
Copy link
Contributor

lnicola commented Jan 30, 2026

@JanssenBrm please see #333

@JanssenBrm
Copy link
Contributor Author

Thank you for updating the logos @lnicola. I have merged your branch into this one so that we can track everything in a single branch.

I did apply a minor change as the reference to the light and dark logos was switched. For the rest, everything seems to be looking great. Feel free to review your records in the preview at https://esa-apex.github.io/apex-algorithms-catalogue-web/pr-preview/pr-321/?page=2. It would be great if you can official approve the content as a comment. After approval, we will merge the records, making them available in our APEx algorithm catalogue.

…s/sen4cap_l3b_biophysical.json

Co-authored-by: Laurențiu Nicola <lnicola@users.noreply.github.com>
…s/sen4cap_l3b_biophysical.json

Co-authored-by: Laurențiu Nicola <lnicola@users.noreply.github.com>
Copy link
Contributor

@lnicola lnicola left a comment

Choose a reason for hiding this comment

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

LGTM overall. I think we'll tweak the wording (and cost structure) a bit when adding the other processors, but it's fine for now.

@lnicola
Copy link
Contributor

lnicola commented Feb 9, 2026

@JanssenBrm do you think we can merge this for now?

@JanssenBrm JanssenBrm merged commit 59d2122 into main Feb 10, 2026
5 checks passed
@JanssenBrm
Copy link
Contributor Author

Merged this and I will trigger a redeploy of the catalogue to visualise the new algorithms. Thank you @lnicola !

@JanssenBrm
Copy link
Contributor Author

@lnicola - The algorithms are now available at https://algorithm-catalogue.apex.esa.int/?page=2

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

Comments