search

ProjectPythia / kerchunk-cookbook

Project Pythia cookbook for Kerchunk
https://projectpythia.org/kerchunk-cookbook/
Apache License 2.0
10 stars 11 forks source link

updated the pre-commit configuration #63

Closed Anu-Ra-g closed 2 weeks ago

Anu-Ra-g commented 2 weeks ago

This PR contains new pre-commit configuration.

github-actions[bot] commented 2 weeks ago

👋 Thanks for opening this PR! The Cookbook will be automatically built with GitHub Actions. To see the status of your deployment, click below. 🔍 Git commit SHA: 38bdc55290020d9e855ea49f428825436fb94e88 ✅ Deployment Preview URL: In Progress

martindurant commented 2 weeks ago

PR #64 is blocked until this is merged, so would appreciate someone having a look here.

martindurant commented 2 weeks ago

Thanks @norlandrhagen .

martindurant commented 2 weeks ago

@Anu-Ra-g : the build following merging this PR failed, please check https://github.com/ProjectPythia/kerchunk-cookbook/actions/runs/10617299100/job/29429761379

martindurant commented 2 weeks ago

(I can access the file from local now, so it might have been an intermittent network effect)

Anu-Ra-g commented 2 weeks ago

@martindurant The same issue has also been raised here in #61. I'm running the notebook associated with this error.

martindurant commented 2 weeks ago

NoCredentials implies that it is trying to authenticate - but the data is public, so anon=True should be in the remote_options.

Anu-Ra-g commented 2 weeks ago

Yeah, I noticed in the notebook, while opening the reference filesystem remote_options are not provided. I'll run it and make the PR.