-
Notifications
You must be signed in to change notification settings - Fork 9
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
Fix notebook paths #59
Conversation
Why is there a permissions error in the binderbot test? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If the binderbot test can pass, I'll approve.
I don't know yet. I will look into it tomorrow |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure if this fixes the problem, but try this...
Ok. This fixed some of the problems. Some notebooks don't run for other reasons:
|
@kmpaul I believe this is because the kernel's working directory is not the mpas folder. Is there some way the working directory can be changed? |
This is a binderbot issue. I reported it in pangeo-gallery/binderbot#28 |
Is there a work-around? Or would moving this to a new repository with the proper structure be best? |
We could pooch-ify it as a workaround. It's a custom colormap with RGB values, so it seems a little silly to make people download it, but it would probably solve the issue for now. |
What about a new repo with a different structure? Is that a better solution? |
If I understand the Github issue Anderson referenced, the problem is that only notebook files will be transferred, rather than all files in the folder, so I'm not sure how a different repo structure would fix that. It seems the author is planning on uploading all files in the next version, so that should solve the problem. |
@bonnland Ah! I see. It doesn't look like there is a time-line for the v1 release. Maybe we can help Yuvi with it. |
Fixes the half-baked #58