Skip to content

Conversation

@asinghvi17
Copy link
Collaborator

  • Switch all references for reproject from GDAL to Proj, improve docstring
  • Add Proj to the docs project
  • Slightly clarify some doc examples

TODOs:

  • Add official tutorials section with Literate examples
  • Add detailed doc page and exploration of cellarea
  • Add docs about the many ways to change the CRS of a raster
  • Merge in Alex's read/write docs PR here
  • Merge in Lazaro's adds docs for resample, MODIS projection #730 resample PR here.

@lazarusA
Copy link
Collaborator

And every method from here should be documented and added to the nav bar.

Screenshot 2024-10-11 at 03 05 26

@rafaqz
Copy link
Owner

rafaqz commented Oct 11, 2024

Can the nav bar just link to their API docs?

@asinghvi17
Copy link
Collaborator Author

Should the cellarea tutorial also show you doing something with it? Maybe computing average precipitation across Denmark?

@asinghvi17
Copy link
Collaborator Author

Can the nav bar just link to their API docs?

It could but that seems like maintenance hell, best to link to a tutorial I think. The menu bar would also get very long...

@asinghvi17
Copy link
Collaborator Author

@rafaqz should we take this local so a preview can be deployed? I would need write access though.

@rafaqz
Copy link
Owner

rafaqz commented Oct 13, 2024

It could but that seems like maintenance hell, best to link to a tutorial I think. The menu bar would also get very long...

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

@rafaqz
Copy link
Owner

rafaqz commented Oct 13, 2024

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

@asinghvi17
Copy link
Collaborator Author

Makes sense - I can cut this PR off here and start a new one with the resample/warp docs I have locally.

@rafaqz
Copy link
Owner

rafaqz commented Oct 13, 2024

Thanks. Just switch it from draft when you need a review

@asinghvi17
Copy link
Collaborator Author

Succeeded by #800

@asinghvi17 asinghvi17 closed this Oct 13, 2024
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.

3 participants