Skip to content

Sentinelhub reorg#127

Merged
simonvb00 merged 12 commits into
mainfrom
sentinelhub-reorg
Jul 22, 2026
Merged

Sentinelhub reorg#127
simonvb00 merged 12 commits into
mainfrom
sentinelhub-reorg

Conversation

@willrayeo

Copy link
Copy Markdown
Collaborator

Description:

PR Checklist:

  • Code is formatted and linted (Python linting action)
  • Kernel spec metadata was correctly set (check_kernelspec action)
  • Tests pass (test-samples action)
  • If applicable, documentation has been updated to reflect changes

@simonvb00

Copy link
Copy Markdown
Collaborator

Hi @willrayeo! I noticed that some SentinelHub samples were failing. Could you fix these in the testing pipeline within your refactoring?

I have identified these ones:
Statistical_API_advanced_visualisations.ipynb > fails due to "config = SHConfig("cdse")" which is a known limitation of the testing pipeline. Normally if you just comment out these lines, the pipeline doesn't fail for these notebooks!

BIDs_2025_Intro_to_SH_APIs.ipynb > Failed to open dataset (flags=68): latvia_fields_wgs84.geojson -, reason unknown.

Thanks a lot!

@willrayeo

Copy link
Copy Markdown
Collaborator Author

thanks Simon, fixing this morning. Can you look into whether the contextily library is importing correctly as I think some of the errors might be related to that.

@willrayeo

Copy link
Copy Markdown
Collaborator Author

@simonvb00 there is inconsitency here, sometimes the test works and sometimes it fails. I cannot replicate the errors myself.

@simonvb00

Copy link
Copy Markdown
Collaborator

Hi @willrayeo, thank you for taking a look at the other tests. If one specific sample has inconsistent results, but works fine on the CDSE Jupyterlab environment, you can remove the specific test from the sentinelhub/.tests file. This way, that specific sample gets excluded from the automated testing.

@willrayeo

Copy link
Copy Markdown
Collaborator Author

@simonvb00 there are failures in the OpenEO tests. However, if you are not concerned then please complete the Pull Request

@simonvb00

Copy link
Copy Markdown
Collaborator

@willrayeo Pratichhya is working on a fix for the OpenEO samples.

@simonvb00
simonvb00 merged commit 36373bd into main Jul 22, 2026
4 of 5 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