Skip to content

BUG: DTA/TDA incorrectly accepting iNaT for setitem #27699

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

Merged
merged 2 commits into from
Aug 2, 2019

Conversation

jbrockmendel
Copy link
Member

  • closes #xxxx
  • tests added / passed
  • passes black pandas
  • passes git diff upstream/master -u -- "*.py" | flake8 --diff
  • whatsnew entry

First of several related PRs.

@jbrockmendel
Copy link
Member Author

@TomAugspurger the Travis failure here seems to be affecting everything. The traceback looks like its somewhere in xdist or pytest. Didn't we recently deal with something really similar?

@TomAugspurger
Copy link
Contributor

TomAugspurger commented Aug 1, 2019 via email

@jreback jreback added Indexing Related to indexing on series/frames, not to indexes themselves Datetime Datetime data dtype Missing-data np.nan, pd.NaT, pd.NA, dropna, isnull, interpolate labels Aug 2, 2019
@jbrockmendel
Copy link
Member Author

green

@jreback jreback added this to the 1.0 milestone Aug 2, 2019
@@ -418,7 +418,7 @@ def test_value_counts_unique_nunique_null(self, null_obj):
values = o._shallow_copy(v)
else:
o = o.copy()
o[0:2] = iNaT
Copy link
Contributor

Choose a reason for hiding this comment

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

same questions as #27704 (comment) (happy to do on either PR)

@jreback jreback merged commit 08a4c1a into pandas-dev:master Aug 2, 2019
@jreback
Copy link
Contributor

jreback commented Aug 2, 2019

thanks

@jbrockmendel jbrockmendel deleted the dtafix branch August 2, 2019 16:03
quintusdias pushed a commit to quintusdias/pandas_dev that referenced this pull request Aug 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Datetime Datetime data dtype Indexing Related to indexing on series/frames, not to indexes themselves Missing-data np.nan, pd.NaT, pd.NA, dropna, isnull, interpolate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants