-
Notifications
You must be signed in to change notification settings - Fork 39
[WIP] More docs #789
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
[WIP] More docs #789
Conversation
|
Can the nav bar just link to their API docs? |
|
Should the cellarea tutorial also show you doing something with it? Maybe computing average precipitation across Denmark? |
It could but that seems like maintenance hell, best to link to a tutorial I think. The menu bar would also get very long... |
|
@rafaqz should we take this local so a preview can be deployed? I would need write access though. |
I mean putting the links to those methods in the sidebar manually. They have pretty extensive documentation with plots etc, best to not do that twice |
|
Also: I'm not sure about the huge dotpoint PRs. Like I like the effort a lot! but it may be better to do what you can, merge it, do a bit more. The checklist can go in an issue? Then it's more democratic and other people can do some of them. PRs that are too big and unfocused get very hard to review |
|
Makes sense - I can cut this PR off here and start a new one with the resample/warp docs I have locally. |
|
Thanks. Just switch it from draft when you need a review |
|
Succeeded by #800 |

TODOs: