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.

Sign up for GitHub

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

Jump to bottom

BUG: Allow TZ-aware DatetimeIndex in merge_asof() (#14844) #14845

Closed
wants to merge 1 commit into from
Closed

BUG: Allow TZ-aware DatetimeIndex in merge_asof() (#14844) #14845

wants to merge 1 commit into from

Conversation

chrisaycock
Copy link
Contributor

@jreback jreback added Bug Reshaping Concat, Merge/Join, Stack/Unstack, Explode Timezones Timezone data dtype labels Dec 9, 2016
@jreback jreback added this to the 0.19.2 milestone Dec 9, 2016
@jreback
Copy link
Contributor

jreback commented Dec 9, 2016

lgtm. ping on green.

@chrisaycock
Copy link
Contributor Author

@jreback So Travis passed but AppVeyor failed because of the one-hour time out. Should I just resubmit?

@jreback
Copy link
Contributor

jreback commented Dec 9, 2016

I restarted the job, but very odd have seen that a lot recently. And some succeed just fine.

@jreback
Copy link
Contributor

jreback commented Dec 10, 2016

thanks!

@jreback jreback closed this in e991141 Dec 10, 2016
@chrisaycock chrisaycock deleted the GH14844 branch December 10, 2016 16:42
jorisvandenbossche pushed a commit that referenced this pull request Dec 15, 2016
@jorisvandenbossche
BUG: Allow TZ-aware DatetimeIndex in merge_asof() (#14844)
7f53ea8 
closes #14844

Author: Christopher C. Aycock <christopher.aycock@twosigma.com>

Closes #14845 from chrisaycock/GH14844 and squashes the following commits:

97b73a8 [Christopher C. Aycock] BUG: Allow TZ-aware DatetimeIndex in merge_asof() (#14844)

(cherry picked from commit e991141)
ischurov pushed a commit to ischurov/pandas that referenced this pull request Dec 19, 2016
@ischurov
BUG: Allow TZ-aware DatetimeIndex in merge_asof() (pandas-dev#14844)
38b52fd 
closes pandas-dev#14844

Author: Christopher C. Aycock <christopher.aycock@twosigma.com>

Closes pandas-dev#14845 from chrisaycock/GH14844 and squashes the following commits:

97b73a8 [Christopher C. Aycock] BUG: Allow TZ-aware DatetimeIndex in merge_asof() (pandas-dev#14844)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Bug Reshaping Concat, Merge/Join, Stack/Unstack, Explode Timezones Timezone data dtype
Projects
None yet
Milestone
0.19.2
Development

Successfully merging this pull request may close these issues.

merge_asof() must be able to operate with timezone-aware DatetimeIndex
2 participants
@chrisaycock @jreback