TST: Extend datetime64 arith tests to array classes, fix several broken cases #23771
Conversation
|
Hello @jbrockmendel! Thanks for updating the PR.
Comment last updated on November 18, 2018 at 21:11 Hours UTC |
| if self.freq is not None: | ||
| return type(self)(-self._data, freq=-self.freq) | ||
| return type(self)(-self._data) | ||
|
|
There was a problem hiding this comment.
This will be made unnecessary by #23642.
| # RelativeDelta DateOffsets | ||
|
|
||
| # ------------------------------------------------------------- | ||
| # Non-Tick, Non-RelativeDelta DateOffsets |
There was a problem hiding this comment.
Some of the tests currently at the bottom of this file will end up in these two sections. For now leaving them in place to make the diff clearer.
jreback
added
Testing
| @@ -791,6 +795,10 @@ def __sub__(self, other): | |||
| else: # pragma: no cover | |||
| return NotImplemented | |||
|
|
|||
| if is_timedelta64_dtype(result) and isinstance(result, np.ndarray): | |||
There was a problem hiding this comment.
Not really. It would just say # wrap result in TimedeltaArray which is pretty redundant given the one-liner here
| pd.offsets.MonthEnd()]) | ||
| exp = klass([Timestamp('2001-1-1'), Timestamp('2000-2-29')]) | ||
| other = tm.box_expected(other, box_with_array) |
There was a problem hiding this comment.
this file getting kind of hefty. ok with making a subdir if it helps, e.g.
pandas/tests/arithmetic/datetime64/test_offsets....etc.
There was a problem hiding this comment.
I'm hopeful we'll be able to de-duplicate a bunch of these (many moved from series/frame/index tests) and slim it down. If not then subdirs may be the way to go.
|
looks good. ping on green. |
Travis fail is the test_apply segfault. gentle ping |
|
thanks |
…fixed * upstream/master: (46 commits) DEPS: bump xlrd min version to 1.0.0 (pandas-dev#23774) BUG: Don't warn if default conflicts with dialect (pandas-dev#23775) BUG: Fixing memory leaks in read_csv (pandas-dev#23072) TST: Extend datetime64 arith tests to array classes, fix several broken cases (pandas-dev#23771) STYLE: Specify bare exceptions in pandas/tests (pandas-dev#23370) ENH: between_time, at_time accept axis parameter (pandas-dev#21799) PERF: Use is_utc check to improve performance of dateutil UTC in DatetimeIndex methods (pandas-dev#23772) CLN: io/formats/html.py: refactor (pandas-dev#22726) API: Make Categorical.searchsorted returns a scalar when supplied a scalar (pandas-dev#23466) TST: Add test case for GH14080 for overflow exception (pandas-dev#23762) BUG: Don't extract header names if none specified (pandas-dev#23703) BUG: Index.str.partition not nan-safe (pandas-dev#23558) (pandas-dev#23618) DEPR: tz_convert in the Timestamp constructor (pandas-dev#23621) PERF: Datetime/Timestamp.normalize for timezone naive datetimes (pandas-dev#23634) TST: Use new arithmetic fixtures, parametrize many more tests (pandas-dev#23757) REF/TST: Add more pytest idiom to parsers tests (pandas-dev#23761) DOC: Add ignore-deprecate argument to validate_docstrings.py (pandas-dev#23650) ENH: update pandas-gbq to 0.8.0, adds credentials arg (pandas-dev#23662) DOC: Improve error message to show correct order (pandas-dev#23652) ENH: Improve error message for empty object array (pandas-dev#23718) ...
Some of the DateOffset arithmetic didn't work with the EA classes; this fixed that. Other pieces of the DateOffset arithmetic will have to wait until TimedeltaArray gets a few more methods.