Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
CustomBusinessDay slow for increments <0 and >1 #8236
Comments
|
hmm, take that back...go ahead and do a PR (and add vbenches) |
jreback
added Performance Frequency
labels
Sep 10, 2014
This was referenced Sep 12, 2014
jreback
added this to the
0.15.1
milestone
Sep 18, 2014
jreback
modified the milestone: 0.15.0, 0.15.1
Oct 4, 2014
jreback
closed this
in #8293
Oct 4, 2014
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
bjonen commentedSep 10, 2014
For rolling back or for moving forward more than one step,
DateOffsets create a new instance settingnaccordingly. TheCustomBusinessDayconstructor can be very slow, however, when a large list ofholidaysis passed. The holidays are processed to datetime64.As a result speed deteriorates by two magnitudes.
I think it makes sense to have the user call
to_dt64before initializingCustomBusinessDayand to rely on the holidays to be passed in the correct format. Any other ideas how to handle this problem? Shall I create a PR for this?