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

coast.rst: -M argument, mention FORMAT_FLOAT_OUT to reduce number of … #8524

Merged
merged 1 commit into from
Jun 19, 2024

Conversation

anbj
Copy link
Contributor

@anbj anbj commented Jun 19, 2024

Address #8516.

@anbj anbj requested review from joa-quim and a team June 19, 2024 13:05
@anbj
Copy link
Contributor Author

anbj commented Jun 19, 2024

Dont approve! Must change link.

@anbj
Copy link
Contributor Author

anbj commented Jun 19, 2024

Ok, good to go.

@@ -219,6 +219,10 @@ Optional Arguments
just certain levels (1-4) then use the full syntax **-W**\ *level*/\ *pen*
and repeat for each level (pen is not used but required to parse the level correctly).

Since the resolution of the datasets typically are in the order
of 100s of meter, consider reducing the number of decimals by
setting :term:`FORMAT_FLOAT_OUT`.
Copy link
Member

Choose a reason for hiding this comment

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

I found "100s" to be a bit confusing. I first tought in 100 sec-arc. I would prefer "hundreds of meters"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Agree - changing.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Please check now.

@Esteban82 Esteban82 merged commit 79033c0 into master Jun 19, 2024
1 check passed
@Esteban82 Esteban82 deleted the coast-doc branch June 19, 2024 13:31
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.

None yet

2 participants