-
Notifications
You must be signed in to change notification settings - Fork 58
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
melt.data.frame no longer barfs with POSIXlt #69
Conversation
@hadley no idea what this error is:
|
|
||
// POSIXlt objects are _weird_ | ||
if (Rf_inherits(object, "POSIXlt")) { | ||
static Function R_rep("rep", "base"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd be tempted to just make this an error
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I feel comfortable throwing errors whenever you see a POSIXlt because:
class(data.frame(as.POSIXlt(Sys.time()))[[1]])
# POSIXct
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay, I'll copy what you did in tidyr
here.
@kevinushey https://github.com/hadley/reshape/blob/master/.travis.yml#L8 needs to be This must have been written back in the old days when Hadley was the default user for devtools. 😄 added PR #70 to fix this |
810f19e
to
6342c11
Compare
@jimhester thanks for looking into that! Ah, the good old days ;) @hadley, I've updated the PR (essentially conforming with what you've done in |
Could you please merge/rebase to get the updated .travis.yml? |
6342c11
to
02f9884
Compare
Done; hopefully all should be well now. |
melt.data.frame no longer barfs with POSIXlt
Thanks! |
No description provided.