-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
ENH: Partial string matching for timestamps with multiindex #12530
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
Conversation
|
||
.. ipython:: python | ||
|
||
dft2 = DataFrame(randn(20,1), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pls use pd.
and np.
, see #9886.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sinhrks, Fixed.
@@ -22,6 +22,8 @@ New features | |||
Enhancements | |||
~~~~~~~~~~~~ | |||
|
|||
Partial string matches on `DateTimeIndex` now work when part of a `MultiIndex` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use double-back ticks around DatetimeIndex
(sp as well) (and same around MultiIndex
); put the issue reference on the same line
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
looks real nice @markroth8 just some minor doc-comments. pls rebase on master when you get a chance; master now builds correctly. |
actually pls also put an example in the this section here as well |
pls rebase and i'll have a look |
@@ -22,6 +22,8 @@ New features | |||
Enhancements | |||
~~~~~~~~~~~~ | |||
|
|||
Partial string matches on `DateTimeIndex` now work when part of a `MultiIndex` | |||
(:issue:`10331`) | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pls add a min-example here (you can add a sub-section if you'd like to highlite to setup an example and show how it works)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you can pretty much use the example for timeseries.rst
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
@markroth8 can you update/rebase. |
Includes implementation, unit tests, documentation fixes and whatsnew.
@jreback: Incorporated suggestions and rebased / pushed. |
git diff upstream/master | flake8 --diff
Includes implementation, unit tests, documentation fixes and whatsnew.