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

Detailed proj_trans print #50

Merged
merged 8 commits into from
Jan 19, 2024
Merged

Detailed proj_trans print #50

merged 8 commits into from
Jan 19, 2024

Conversation

anthonynorth
Copy link
Collaborator

Adds detailed proj transformer print method. Example:

PROJ::proj_trans_create("EPSG:4283", "EPSG:3112")
#> <proj_trans at 0xaaab00fccfa0>
#> type: Concatenated Operation
#> id: pipeline
#> description: axis order change (2D) + Geoscience Australia Standard National Scale Lambert Projection
#> definition: proj=pipeline step proj=unitconvert xy_in=deg xy_out=rad step proj=lcc lat_0=0 lon_0=134 lat_1=-18 lat_2=-36 x_0=0 y_0=0 ellps=GRS80
#> area_of_use:
#>   name: World
#>   bounds: [-180 -90 180 90]
#> source_crs:
#>   type: Geographic 2D CRS
#>   id: EPSG:4283
#>   name: GDA94
#>   area_of_use:
#>     name: Australia including Lord Howe Island, Macquarie Island, Ashmore and Cartier Islands, Christmas Island, Cocos (Keeling) Islands, Norfolk Island. All onshore and offshore.
#>     bounds: [93.41 -60.55 173.34 -8.47]
#> target_crs:
#>   type: Projected CRS
#>   id: EPSG:3112
#>   name: GDA94 / Geoscience Australia Lambert
#>   area_of_use:
#>     name: Australia - Australian Capital Territory; New South Wales; Northern Territory; Queensland; South Australia; Tasmania; Western Australia; Victoria.
#>     bounds: [112.85 -43.7 153.69 -9.86]

Created on 2024-01-17 with reprex v2.0.2

@anthonynorth
Copy link
Collaborator Author

@mdsumner, is the print output too verbose? Or is anything missing that you'd like added?

Normalising has some affect on the area_of_use. In this example, area of use for the not-normalised transformer is [112.85 -43.7 153.69 -9.86] instead of World. We can patch this if it's important.

@mdsumner
Copy link
Member

all the information I want to see is there 🙏🤘

Copy link
Member

@mdsumner mdsumner left a comment

Choose a reason for hiding this comment

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

looks good to me!

@mdsumner mdsumner merged commit 7fb4b30 into hypertidy:main Jan 19, 2024
6 of 7 checks passed
@anthonynorth anthonynorth deleted the wk-print branch January 19, 2024 00:32
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.

2 participants