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

Concerns about using "secondly" and "minutely" as period values #8

Closed
timburks opened this issue Dec 10, 2020 · 3 comments · Fixed by #13
Closed

Concerns about using "secondly" and "minutely" as period values #8

timburks opened this issue Dec 10, 2020 · 3 comments · Fixed by #13

Comments

@timburks
Copy link
Contributor

timburks commented Dec 10, 2020

While "hourly", "daily", "weekly", and "monthly" all are understood to mean "per unit of time", "secondly" and "minutely" have different primary meanings (though I have found one definition consistent with this usage).

Alternately, "second", "minute", etc. are well-understood as quantities of time, which is consistent with the definition of "period": "a length or portion of time". These also are simpler words because they don't have the extra "-ly" suffix. (And I think words like "monthly" are commonly considered "frequencies" vs periods).

Based on this, I'd suggest using the time quantities "second", "minute", etc. as period values in the limit object with the possible addition of support for integers that specify seconds, e.g. "3600" as an alternative to "hourly".

@pjmolina
Copy link
Contributor

+1 I agree with you @timburks. Using simpler words make it easy to use.

We can discuss it in the next SIG meting and take a decision on this. You are more than invited to join if you can.

@pjmolina
Copy link
Contributor

pjmolina commented Dec 17, 2020

Consider https://en.wikipedia.org/wiki/ISO_8601 for periods of time.

@pafmon
Copy link
Contributor

pafmon commented Dec 18, 2020

Incorporated on #13.

@pafmon pafmon closed this as completed Dec 18, 2020
@pjmolina pjmolina linked a pull request Dec 18, 2020 that will close this issue
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 a pull request may close this issue.

3 participants