Skip to content

Add earthdaily methane detection#511

Open
JanssenBrm wants to merge 7 commits into
mainfrom
add-earthdaily-methane-detection
Open

Add earthdaily methane detection#511
JanssenBrm wants to merge 7 commits into
mainfrom
add-earthdaily-methane-detection

Conversation

@JanssenBrm
Copy link
Copy Markdown
Contributor

No description provided.

@algorithm-services-catalogue
Copy link
Copy Markdown

algorithm-services-catalogue Bot commented May 26, 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-511/

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

@JanssenBrm
Copy link
Copy Markdown
Contributor Author

@raster-blaster - You can now review the record at https://esa-apex.github.io/apex-algorithms-catalogue-web/pr-preview/pr-511/?q=methane. Some feedback:

  • The logos for light and dark for both the provider and the platforms are the same, causing some visualisation issues on the catalogue
  • There is currently no execution link where the user can try out the OGC API service. Is there any execution interface (e.g. EODASH instance) or notebook example that showcases how to use the OGC API endpoint? Such a link can be added by using the webapp and/or notebook relationships.
  • There is no link for the user to get access to the service. You can use the order relationship for this purpose.

@raster-blaster
Copy link
Copy Markdown

Hi @JanssenBrm, thank you for continuing the PR and applying the additional fixes.

I addressed the remaining catalogue feedback on my fork branch:

Branch:
https://github.com/raster-blaster/apex_algorithms/tree/add-earthdaily-methane-detection

Commit:
f00a19b

Changes included:

Could you please review these changes when you get a chance?

Thank you.

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