Skip to content

Comments

FIX: Added spyndex dep#173

Merged
ciaransweet merged 2 commits intomainfrom
spyndex_dep
Feb 19, 2026
Merged

FIX: Added spyndex dep#173
ciaransweet merged 2 commits intomainfrom
spyndex_dep

Conversation

@davemlz
Copy link
Contributor

@davemlz davemlz commented Feb 17, 2026

No description provided.

@ciaransweet
Copy link
Contributor

Thanks for reopening!

Out of interest, why do we need to add this? (I've not noticed a breaking notebook because we didn't have it)

@davemlz
Copy link
Contributor Author

davemlz commented Feb 18, 2026

@ciaransweet
Copy link
Contributor

ciaransweet commented Feb 18, 2026

You're welcome!

Yes, this one breaks: https://github.com/eopf-toolkit/eopf-101/blob/main/06_eopf_zarr_in_action/68_vegetation_anomalies.ipynb

Job: https://github.com/eopf-toolkit/eopf-101/actions/runs/22097999374/job/63860026993?pr=171

Ah, that PR also enables it, hence why our nightly run hasn't picked up the error! Great, will approve 🫡

@davemlz could you run uv sync in the root of the repo too? and commit the changes to the uv.lock?

If you don't have uv setup let me know and I can do that.

@davemlz
Copy link
Contributor Author

davemlz commented Feb 19, 2026

I don't have uv setup, if you can do it that would be great! Thanks!

Copy link
Collaborator

@gisromerocandanedo gisromerocandanedo left a comment

Choose a reason for hiding this comment

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

Thanks to both @davemlz and @ciaransweet

Now i understand better how is everything connected :D

@ciaransweet ciaransweet merged commit a3efd5e into main Feb 19, 2026
3 checks passed
@ciaransweet ciaransweet deleted the spyndex_dep branch February 19, 2026 09:57
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.

3 participants