Skip to content

BUG/API: to_timedelta unit-argument ignored for string input #34379

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.

Sign up for GitHub

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

Closed
wants to merge 53 commits into from
Closed

BUG/API: to_timedelta unit-argument ignored for string input #34379

Changes from 1 commit
Commits
Show all changes
53 commits
Select commit Hold shift + click to select a range
ab5f7ba
Added null context manager as do_not_raise for parametrization in pyt…
Oct 5, 2018
7a38ecd
Added test for pure integer strings and test for strings with/without…
Oct 5, 2018
a65a35a
Added entry to whatsnew.
Oct 5, 2018
57dea1d
Changed Timedelta to raise for strings without unit.
Oct 5, 2018
b8cf952
WIP refactoring.
Oct 7, 2018
8154859
Deprecation warning when we don't have units in string numbers.
Oct 7, 2018
a90a992
Fixed linting.
Oct 7, 2018
1184b6e
Added to_timedelta test.
Oct 7, 2018
5e28b49
scalar case in to_timedelta checks for strings.
Oct 7, 2018
fafe838
Updated test with more cases for to_timedelta.
Oct 7, 2018
3ca0dae
All tests passing. Implemented checks on parse_timedelta_string.
Oct 7, 2018
8105f9c
Updated tests to avoid creating timedeltas from strings without unit.
Oct 7, 2018
e90a18b
Updated more tests to avoid creating timedeltas from strings without …
Oct 7, 2018
e0a55a8
Updated more tests to avoid creating timedeltas from strings without …
Oct 7, 2018
827aebf
Fixed failing test.
Oct 7, 2018
9bff504
Removed litle whitespace.
Oct 8, 2018
b6c7d45
Type specified unit as object.
Oct 8, 2018
9ff6ade
Removed newline.
Oct 8, 2018
be93e4b
Moved default unit down to more core functions.
Oct 10, 2018
85f0103
Fixed linting.
Oct 10, 2018
673ad19
Merge remote-tracking branch 'upstream/master' into bugfix/string-tim…
Oct 23, 2018
eb7d3f8
Merge remote-tracking branch 'origin/bugfix/string-timedelta2' into b…
Oct 23, 2018
bb40f77
Merge remote-tracking branch 'upstream/master' into bugfix/string-tim…
Oct 24, 2018
2e331c1
Merge remote-tracking branch 'upstream/master' into bugfix/string-tim…
Oct 31, 2018
bada4a7
Minor typo.
Nov 5, 2018
8c0425a
Lint
Nov 5, 2018
493a14b
Using ValueError instead of Exception.
Nov 5, 2018
e525ec4
Merge remote-tracking branch 'upstream/master' into bugfix/string-tim…
Nov 5, 2018
448b68a
Catching TypeError as well.
Nov 5, 2018
bd173a9
LINT
Nov 5, 2018
d16c4c5
Merge remote-tracking branch 'upstream/master' into bugfix/string-tim…
Nov 6, 2018
1cfc729
Refactored tests, fixed little bug in array_to_timedelta64 and explai…
Nov 6, 2018
e559f06
Fixed unit as None (instead of ns), Fixed test.
Nov 6, 2018
b914950
LINT: Fixed long lines.
Nov 10, 2018
64a4440
LINT: Fixed long lines.
Nov 10, 2018
4d26474
LINT: Fixed indentation of closing bracket.
Nov 10, 2018
db41054
Merge remote-tracking branch 'upstream/master' into bugfix/string-tim…
Nov 26, 2018
2f53004
Added comment.
Nov 26, 2018
fb40c94
Merge remote-tracking branch 'upstream/master' into bugfix/string-tim…
Nov 26, 2018
f5e59ee
Casting unit to 'ns' if None.
Nov 26, 2018
e7266da
Updated documentation and issuing DeprecationWarning instead of excep…
Nov 26, 2018
b6a02df
Fixed linting.
Nov 26, 2018
694abe9
DOC: Reorganized whatsnew
Nov 27, 2018
4d86ae1
CLN: Minor refactoring.
Nov 27, 2018
40b09f4
Merge remote-tracking branch 'upstream/master' into bugfix/string-tim…
Nov 27, 2018
a9c4675
Merge remote-tracking branch 'upstream/master' into bugfix/string-tim…
Nov 27, 2018
95302de
Merge remote-tracking branch 'upstream/master' into bugfix/string-tim…
Dec 3, 2018
661fd70
Merge remote-tracking branch 'upstream/master' into bugfix/string-tim…
Jan 7, 2019
80a1161
Added default value to func declaration.
Jan 8, 2019
dbe8fae
Fixed bug in warning string.
Jan 8, 2019
5a0812c
Merge branch 'master' into bugfix/string-timedelta2
pdnm May 26, 2020
b8031c2
Minor fix
pdnm May 27, 2020
cd17104
Merge branch 'master' into bugfix/string-timedelta2
pdnm May 29, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Catching TypeError as well.
  • Loading branch information
Victor Maryama committed Nov 5, 2018
commit 448b68a30f49ced9c0295118b46169353dc6b33b
2 changes: 1 addition & 1 deletion pandas/core/tools/timedeltas.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -145,7 +145,7 @@ def _coerce_scalar_to_timedelta_type(r, unit=None, box=True, errors='raise'):
try:
result = parse_timedelta_string(r, unit)
result = np.timedelta64(result)
except ValueError:
except (ValueError, TypeError):
try:
result = convert_to_timedelta64(r, unit)
except ValueError:
Expand Down