-
-
Notifications
You must be signed in to change notification settings - Fork 71
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
split.xts() does not return named list #357
Comments
Thanks for the report! I agree Do you expect the names to be You can disable the TZ warnings globally with |
Perhaps split.xts could have a tz= argument to specify this.
…On Wed, Oct 6, 2021 at 8:35 AM Joshua Ulrich ***@***.***> wrote:
Thanks for the report! I agree split.xts() should be consistent with zoo
and return a named list.
You can disable the TZ warnings globally with options(xts_check_TZ =
FALSE). I'll look into why the warning appears here and see if I can
quiet it.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#357 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AB32F7V35AYHZJMPDZT36KTUFQ7B3ANCNFSM5FOMRRNA>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
--
Statistics & Software Consulting
GKX Group, GKX Associates Inc.
tel: 1-877-GKX-GROUP
email: ggrothendieck at gmail.com
|
What do you suggest for the names? The value of the first index value, formatted to the periodicity? E.g. "days" -> Date; "months" -> yearmon; "quarters" -> yearqtr; "years" -> "%Y"? |
Seems goo
…On Fri, Oct 8, 2021 at 9:01 AM Joshua Ulrich ***@***.***> wrote:
What do you suggest for the names? The value of the first index value,
formatted to the periodicity? E.g. "days" -> Date; "months" -> yearmon;
"quarters" -> yearqtr; "years" -> "%Y"?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#357 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AB32F7SHKJCIGLLGQIRJ7MTUF3TSLANCNFSM5FOMRRNA>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
--
Statistics & Software Consulting
GKX Group, GKX Associates Inc.
tel: 1-877-GKX-GROUP
email: ggrothendieck at gmail.com
|
I looked into this a bit. The warning is thrown by
You need to change the xts object's tzone if you want to split on days in a different timezone. For example: I'll name the returned list, but I don't think |
split.xts
should give a named list rather than an unnamed list in the example below. Also it gives an annoying warning message and there is no way to turn it off or control the time zone with respect to which it is split short of usingsplit(x, as.Date(time(x)))
orsplit(x, as.Date(time(x), tz = whatever))
giving:
The text was updated successfully, but these errors were encountered: