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

More tests verifying the output of the cct utility #2465

Closed
wants to merge 1 commit into from
Closed

More tests verifying the output of the cct utility #2465

wants to merge 1 commit into from

Conversation

ghost
Copy link

@ghost ghost commented Nov 28, 2020

On request more tests have been added verifying the output of
the cct utility.

cct has an option (-d) which overrides the default value for
the number of decimals (precision) that are printed when cct
outputs the value of a coordinate.

By default the arguments to the printf format string will be:

  • radians, degrees: width 14, precision 10
  • meters: width 13, precision 4 [1]

[1] height will be shown with a width of 12.

Note: the 4th "coordinate", time, will always be shown with
a precision of 4 decimals (and width 12).

When overridden (using the -d option) all coordinates, but
not time, will be shown with the specified decimals.

  • Closes #xxxx
  • Tests added
  • Added clear title that can be used to generate release notes
  • Fully documented, including updating docs/source/*.rst for new API

On request more tests have been added verifying the output of
the cct utility.

cct has an option (-d) which overrides the default value for
the number of decimals (precision) that are printed when cct
outputs the value of a coordinate.

By default the arguments to the printf format string will be:

 * radians, degrees: width 14, precision 10
 * meters: width 13, precision 4 [1]

[1] height will be shown with a width of 12.

Note: the 4th "coordinate", time, will always be shown with
a precision of 4 decimals (and width 12).

When overridden (using the -d option) all coordinates, but
not time, will be shown with the specified decimals.
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.

1 participant