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

as.period(interval(...)) off by a month #371

Closed
tjmahr opened this issue Jan 19, 2016 · 1 comment
Closed

as.period(interval(...)) off by a month #371

tjmahr opened this issue Jan 19, 2016 · 1 comment
Labels
bug an unexpected problem or unintended behavior

Comments

@tjmahr
Copy link

tjmahr commented Jan 19, 2016

packageVersion("lubridate")
#> [1] ‘1.5.0’

t1 <- "1985-11-24"
t2 <- "2007-08-13"
as.period(interval(t1, t2))
#> [1] "21y 9m 19d 23H 0M 0S"

But the amount of time between the two dates is 21 years, 8 months, 20 days.

@vspinu
Copy link
Member

vspinu commented Jan 20, 2016

This is a bug. Wrong computation with months.I am a bit surprised this went though unnoticed in our tests.

@vspinu vspinu added the bug an unexpected problem or unintended behavior label Jan 20, 2016
@vspinu vspinu closed this as completed in d6eb225 Mar 6, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug an unexpected problem or unintended behavior
Projects
None yet
Development

No branches or pull requests

2 participants