/pandas

New issue

ERR: between_time should raise on non-timelike objects #11818

Closed
jreback opened this Issue Dec 11, 2015 · 1 comment

Comments

Assignees
No one assigned
Labels
Difficulty Novice Effort Low Error Reporting Timeseries
Projects
None yet
Milestone
  0.18.0
1 participant
@jreback
@jreback
Contributor

jreback commented Dec 11, 2015

from SO

so passing anything except time objects or strings should raise (might be a tad tricky as we don't have great time parsers, they all parse datetimelikes).

IIRC this was done for convenience at the time. But can lead to an unstatisfactory user experience.

@jreback jreback added Timeseries Error Reporting labels Dec 11, 2015

jreback added this to the Next Major Release milestone Dec 11, 2015

@jreback jreback added Difficulty Novice Effort Low labels Dec 11, 2015

rockg referenced this issue Dec 12, 2015

Closed

ERR: between_time now checks for argument types #11832

@jreback jreback modified the milestone: 0.18.0, Next Major Release Dec 13, 2015

@jreback jreback added a commit that referenced this issue Jan 3, 2016

@rockg @jreback rockg + jreback ERR/ENH: between_time checks argument types and new to_time function, #… 
…11818
 9c71dbf
@jreback
Contributor

jreback commented Jan 3, 2016

closed by #11832

jreback closed this Jan 3, 2016

jreback referenced this issue Mar 21, 2016

Closed

Issue passing a Timestamp to Series.between_time() #12680

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment