Skip to content

Add mangrove mapping workflow#584

Merged
dfguerrerom merged 3 commits intoopenforis:mainfrom
eriklindquist:add-mangrove-mapping-workflow
Apr 8, 2026
Merged

Add mangrove mapping workflow#584
dfguerrerom merged 3 commits intoopenforis:mainfrom
eriklindquist:add-mangrove-mapping-workflow

Conversation

@eriklindquist
Copy link
Copy Markdown
Contributor

Added workflow for mangrove mapping

@dfguerrerom
Copy link
Copy Markdown
Collaborator

@eriklindquist , I'm not sure why RTD is not automatically creating the preview, but FYI, this is automatic build created after your commit https://sepal-doc--584.org.readthedocs.build/en/584/workflows/mangrove_mapping.html

@dfguerrerom
Copy link
Copy Markdown
Collaborator

Hi @eriklindquist — the RTD build failure is not related to your changes. The sphinxcontrib-youtube package recently bumped its minimum Sphinx requirement to >=7.2, which conflicts with the Sphinx<=7.1.2 pin we had in requirements.txt. We've already fixed this on main (#585), but since your branch was created before that fix, RTD picks up the old pin.

Everything else looks good — we'll go ahead and merge. The conflict will resolve automatically once your changes land on main.

@dfguerrerom dfguerrerom merged commit c17e9eb into openforis:main Apr 8, 2026
2 of 3 checks passed
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