-
Notifications
You must be signed in to change notification settings - Fork 401
DOCS: Update solar #8702
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
DOCS: Update solar #8702
Conversation
|
Don't know why you want to change it. It's day/night, not day/light. |
We worked on the PyGMT docs and came cross that both "day/night" and "day/light" are used (GenericMappingTools/pygmt#3857 (comment)) in the GMT docs, please see https://docs.generic-mapping-tools.org/dev/solar.html. |
|
I don't really case if it is |
Hm. I aggree that, using "day-night" and "day-light" may be confusing for users. However, at the beginning of the GMT docs (https://docs.generic-mapping-tools.org/dev/solar.html) there is currently written:
So, I am a unsure how to continue here (or close this PR?). At least the formulations should be consistent across the GMT and PyGMT docs. |
|
Maybe we should just go with |
Honestly, I don't find it so confusing (and that I know, no one complained) but yes, that entry line is better stated as |
|
Now I changed "day-light" consistently to "day-night". |
Description of proposed changes
Relate to GenericMappingTools/pygmt#3857 (comment)
Reminders