Skip to content
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

Some improvements to the docs for tmerc and omerc #1281

Merged
merged 8 commits into from
Feb 24, 2019
Merged

Some improvements to the docs for tmerc and omerc #1281

merged 8 commits into from
Feb 24, 2019

Conversation

busstoptaktik
Copy link
Member

Over at #523, @ralphtee gave some very nice explanations for omerc. I have added some of this material + a some descriptional text to the docs for omerc.

Additionally clarified a few things about algorithms in the docs for tmerc.

Copy link
Member

@kbevers kbevers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall this is very good. I would suggest only keeping the first paragraph ("The Oblique Mercator projection is a cylindrical map projection that closes the
gap between the Mercator and the Transverse Mercator projections") above the classification table and moving the rest of the intro-text below the table. This conforms to the style that is used in most other properly documented projection as well as giving a quick overview of the table without scrolling down the page.

There's some good figures in #523. I don't know the source of the figures but if we are allowed to include them (or reproductions of them) I think they would help illustrate some of the points made about he different types of rotations that can be applied. It's probably a job for another day but at least it is worth considering this as a future improvement.

@kbevers
Copy link
Member

kbevers commented Feb 22, 2019

The updates to omerc and tmerc are not related, right? In that case they would be better placed in separate commits instead of the same.

@busstoptaktik
Copy link
Member Author

The updates to omerc and tmerc are not related, right?

Not as related as I expected when starting the work: While omerc clearly being mostly in need of a touch up, I thought of it as an occasion for a general overhaul of omerc, tmerc, and merc, giving an overall better feeling that they are in close family, although looking much different.

I think I got part of that message through in the intro to omerc, but I ran short of time before getting through everything.

So, yes, it was the idea to make changes logically tied across all 3 projections, but that is probably not terribly evident from this PR.

author = {Mikael Rittri},
title = {New omerc approximations of Denmark System 34},
year = {2012},
url = {http://lists.maptools.org/pipermail/proj/2012-June/006323.html}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you change this to https://lists.osgeo.org/pipermail/proj/2012-June/005926.html , since maptools.org is probably going to be retired at some point

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for noticing this - I had planned to change it, but forgot about it

@kbevers kbevers merged commit 38a1525 into OSGeo:master Feb 24, 2019
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