-
-
Notifications
You must be signed in to change notification settings - Fork 35
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
Cross post dask detrending from xarray blog #176
Conversation
✅ Deploy Preview for dask-blog ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
According to the logs, the build is failing because it needs ruby >=3.0.0 for some of the dependencies to install correctly. We could edit this line of the gemfile to try and fix that. (I think currently Line 2 in 0d3d6e1
|
Thanks for cross-posting here @scharlottej13. And thanks for investigating the build failure @GenevieveBuckley. I've pulled in the build fix from #177, so things should build successfully now. |
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.
Looks like some of the figures are rendering a bit big. Could you take a look at set a maximum width @scharlottej13?
You might need to use HTML <img src="..." width="100%" />
or something along those lines instead of the markdown syntax for images.
Thanks @jacobtomlinson and @GenevieveBuckley! I think these image sizes look much more reasonable |
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.
Thanks @scharlottej13
(And thanks @GenevieveBuckley @jacobtomlinson for reviewing)
Cross-posting @phofl's xarray blog post.