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

ENH: Added parameter output_axis_rule to CRS.to_wkt #1287

Merged
merged 8 commits into from May 19, 2023

Conversation

jjimenezshaw
Copy link
Contributor

@jjimenezshaw jjimenezshaw commented May 16, 2023

This PR adds the parameter output_axis_rule to the method CRS.to_wkt, as documented in https://proj.org/development/reference/functions.html#c.proj_as_wkt

YES is mapped to True,
NO to False,
and AUTO to None (default).

  • Tests added
  • Fully documented, including history.rst for all changes and api/*.rst for new API

pyproj/_crs.pyx Outdated Show resolved Hide resolved
@codecov
Copy link

codecov bot commented May 16, 2023

Codecov Report

Merging #1287 (3180e72) into main (3d0fb75) will not change coverage.
The diff coverage is 100.00%.

❗ Current head 3180e72 differs from pull request most recent head 2893be4. Consider uploading reports for the commit 2893be4 to get more accurate results

@@           Coverage Diff           @@
##             main    #1287   +/-   ##
=======================================
  Coverage   96.41%   96.41%           
=======================================
  Files          20       20           
  Lines        1812     1812           
=======================================
  Hits         1747     1747           
  Misses         65       65           
Impacted Files Coverage Δ
pyproj/crs/crs.py 98.08% <100.00%> (ø)

@snowman2 snowman2 changed the title Add output axis rule ENH: Added parameter output_axis_rule to CRS.to_wkt May 16, 2023
@snowman2 snowman2 added this to the 3.6.0 milestone May 18, 2023
@snowman2 snowman2 merged commit 4348437 into pyproj4:main May 19, 2023
21 checks passed
@snowman2
Copy link
Member

Thanks @jjimenezshaw 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants