Skip to content

TST: Test DatetimeIndex weekend offset #14853

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

Conversation

mroeschke
Copy link
Member

@mroeschke mroeschke commented Dec 10, 2016

closes #5348
closes #5115
closes #5152

The original issues pass on master as of 0.19.1. These related issues might have been fixed part of #5004, but it doesn't look like that any PRs fixed this recently.

result = idx1.append(idx2)
expected = DatetimeIndex([d1, d2, d3])
tm.assert_index_equal(result, expected)
# GH 5115
Copy link
Contributor

Choose a reason for hiding this comment

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

blank line

d3 = date(2012, 9, 30)
idx1 = DatetimeIndex([d1, d2])
idx2 = DatetimeIndex([d3])
result = idx1.append(idx2)
Copy link
Contributor

Choose a reason for hiding this comment

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

the original operation was a .union (but testing append is ok too), so add a test for .union

@jreback jreback added Frequency DateOffsets Testing pandas testing functions or related to the test suite Datetime Datetime data dtype labels Dec 10, 2016
@mroeschke mroeschke force-pushed the test_datetimeindex_frequencies branch from 1153c86 to 736ebb2 Compare December 10, 2016 22:10
@codecov-io
Copy link

codecov-io commented Dec 11, 2016

Current coverage is 85.26% (diff: 100%)

Merging #14853 into master will increase coverage by <.01%

@@             master     #14853   diff @@
==========================================
  Files           144        144          
  Lines         50981      50981          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits          43469      43470     +1   
+ Misses         7512       7511     -1   
  Partials          0          0          

Powered by Codecov. Last update b6de920...736ebb2

@jorisvandenbossche jorisvandenbossche added this to the 0.20.0 milestone Dec 11, 2016
@jorisvandenbossche jorisvandenbossche merged commit 1b0fecd into pandas-dev:master Dec 11, 2016
@jorisvandenbossche
Copy link
Member

@mroeschke Thanks a lot, this is really useful for cleaning up the issues!

yarikoptic added a commit to neurodebian/pandas that referenced this pull request Dec 12, 2016
* origin/master: (22 commits)
  BUG: astype falsely converts inf to integer (GH14265) (pandas-dev#14343)
  BUG: Apply min_itemsize to index even when not appending
  DOC: warning section on memory overflow when joining/merging dataframes on index with duplicate keys (pandas-dev#14788)
  BLD: missing - on secure
  BLD: new access token on pandas-dev
  TST: Test DatetimeIndex weekend offset (pandas-dev#14853)
  BLD: escape GH_TOKEN in build_docs
  TST: Correct results with np.size and crosstab (pandas-dev#4003) (pandas-dev#14755)
  Frame benchmarking sum instead of mean (pandas-dev#14824)
  CLN: lint of test_base.py
  BUG: Allow TZ-aware DatetimeIndex in merge_asof() (pandas-dev#14844)
  BUG: GH11847 Unstack with mixed dtypes coerces everything to object
  TST: skip testing on windows for specific formatting which sometimes hangs (pandas-dev#14851)
  BLD: try new gh token for pandas-docs
  CLN/PERF: clean-up of the benchmarks (pandas-dev#14099)
  ENH: add timedelta as valid type for interpolate with method='time' (pandas-dev#14799)
  DOC: add section on groupby().rolling/expanding/resample (pandas-dev#14801)
  TST: add test to confirm GH14606 (specify category dtype for empty) (pandas-dev#14752)
  BLD: use org name in build-docs.sh
  BF(TST): use = (native) instead of < (little endian) for target data types (pandas-dev#14832)
  ...
ischurov pushed a commit to ischurov/pandas that referenced this pull request Dec 19, 2016
@mroeschke mroeschke deleted the test_datetimeindex_frequencies branch December 21, 2016 04:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Datetime Datetime data dtype Frequency DateOffsets Testing pandas testing functions or related to the test suite
Projects
None yet
4 participants