Skip to content

Conversation

jbrockmendel
Copy link
Member

@jbrockmendel jbrockmendel commented Dec 18, 2019

Copy link
Contributor

@jreback jreback left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tiny comment, can merge and change in a followup if you prefer

PerformanceWarning,
)
result = self.astype("O") + offset
if len(self) == 0:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

use not len

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

follow-up, ive got a branch specifically collecting these things

@jreback jreback added Frequency DateOffsets Timezones Timezone data dtype labels Dec 19, 2019
@jreback jreback added this to the 1.0 milestone Dec 19, 2019
@simonjayhawkins simonjayhawkins merged commit 66038e9 into pandas-dev:master Dec 20, 2019
@jbrockmendel jbrockmendel deleted the add-offsets branch December 20, 2019 16:31
jbrockmendel added a commit to jbrockmendel/pandas that referenced this pull request Dec 25, 2019
AlexKirko pushed a commit to AlexKirko/pandas that referenced this pull request Dec 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Frequency DateOffsets Timezones Timezone data dtype
Projects
None yet
Development

Successfully merging this pull request may close these issues.

BUG: Cannot add non-vectorized DateOffset to empty DatetimeIndex
3 participants