Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

auto dask example #59

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

raybellwaves
Copy link
Contributor

No description provided.

Copy link

github-actions bot commented Apr 5, 2024

👋 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: d602ab8
✅ Deployment Preview URL: https://ProjectPythia.github.io/kerchunk-cookbook/_preview/59

@norlandrhagen
Copy link
Collaborator

Hey there @raybellwaves, thanks for opening up this PR. A few suggestions and we can merge it!

  • Could you add the .md file location to the _toc.yml file so it appears in the table of contents.
  • Most of the chapters have Overview and Prerequisites sections that give the chapter a little bit on context. It might help explaining a bit about Kerchunk's auto_dask, when to use it, some potential stumbling blocks etc.

Thanks for the contribution!

@raybellwaves
Copy link
Contributor Author

Thanks for the feedback @norlandrhagen . Sorry I won't be able to work on this any more. Happy to review though if someone else picks this up.

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