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
Re-organise some coord docs #6954
Conversation
Is this intended to address #6864 as well? |
Nope |
Co-authored-by: Stuart Mumford <stuart@cadair.com>
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.
I am happy with this as it is, but the 2D-3D rsun thing needs to be put somewhere else before we merge it.
I'm unlikely to get back to this in the next week, so if anyone wants to push to my branch feel free |
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 for taking this on @dstansby. To summarize my requests from my inline comments:
- Move "getting started" section to a "how-to" guide on how to create coordinate objects
- Move "transforming between coordinate frames" to a how-to guide
- Move text on the 2D-3D transformation to a topic guide
at this point, I think we could directly copy the text here to those new sections and we can wordsmith/refine in subsequent PRs.
Ok I've addressed the points I highlighted above. I think the RSUN topic guide needs more detail, but I think we can leave that to another PR. |
Fixes #6913.
A lot of the tutorial content was duplicated. In the guide page, there was also some accessing coordinate components content that felt like a how to guide, so I moved that there. I'm open to a bit of bikeshedding in this PR, but would be good to get this in realtively quickly in time for 5.0.