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
BUG: show time values in repr of high precision DatetimeIndex #19109
Conversation
@@ -883,6 +883,20 @@ def test_datetimelike_frame(self): | |||
'[10 rows x 2 columns]') | |||
assert repr(df) == expected | |||
|
|||
def test_datetimeindex_highprecision(self): |
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.
see if you can parametrize this checking each digit of precision
doc/source/whatsnew/v0.23.0.txt
Outdated
@@ -374,7 +374,8 @@ Conversion | |||
- Bug in :class:`TimedeltaIndex` where division by a ``Series`` would return a ``TimedeltaIndex`` instead of a ``Series`` (issue:`19042`) | |||
- Bug in :class:`Series` with ``dtype='timedelta64[ns]`` where addition or subtraction of ``TimedeltaIndex`` could return a ``Series`` with an incorrect name (issue:`19043`) | |||
- Fixed bug where comparing :class:`DatetimeIndex` failed to raise ``TypeError`` when attempting to compare timezone-aware and timezone-naive datetimelike objects (:issue:`18162`) | |||
- | |||
- Bug in :class:`DatetimeIndex` where the repr was not showing the time values for the end of the day (:issue:`19030`) |
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.
make this more clear
Codecov Report
@@ Coverage Diff @@
## master #19109 +/- ##
==========================================
+ Coverage 91.51% 91.53% +0.02%
==========================================
Files 148 148
Lines 48753 48753
==========================================
+ Hits 44616 44628 +12
+ Misses 4137 4125 -12
Continue to review full report at Codecov.
|
338c441
to
055e0ac
Compare
Hello @reidy-p! Thanks for updating the PR. Cheers ! There are no PEP8 issues in this Pull Request. 🍻 Comment last updated on January 07, 2018 at 12:38 Hours UTC |
thanks @reidy-p |
git diff upstream/master -u -- "*.py" | flake8 --diff
On master:
On my branch:
It seems that it's only a problem when the time value was 23:59:59.999999999 or very similar, but I may need to add some more tests to check this further.