Skip to content

Separate parsing functions out from tslib #17363

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

Merged
merged 20 commits into from
Sep 26, 2017
Merged

Separate parsing functions out from tslib #17363

Changes from 1 commit
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
b0428ec
Separate parsing functions out from tslib
jbrockmendel Aug 28, 2017
b19a31e
flake8 whitespace fixup
jbrockmendel Aug 28, 2017
fa6def5
Address reviewer comments
jbrockmendel Aug 29, 2017
2cd2ab6
Fix __doc__ cython complaint
jbrockmendel Aug 29, 2017
c765e26
lint fixup
jbrockmendel Aug 30, 2017
b3ca3c0
Reviewer comments; remove cython decorators
jbrockmendel Sep 6, 2017
e36f8be
kludge to move tslibs/parsing to make asv work
jbrockmendel Sep 6, 2017
c52c796
kludge to move tslibs/parsing to make asv work
jbrockmendel Sep 6, 2017
f5259ea
flake8 whitespace fixup
jbrockmendel Sep 7, 2017
d1765ab
Dummy commit to force CI
jbrockmendel Sep 7, 2017
ceac008
Merge branch 'master' of https://github.com/pandas-dev/pandas into ts…
jbrockmendel Sep 8, 2017
e3995be
Whitespace fixup
jbrockmendel Sep 8, 2017
6db3e3c
rebase
jbrockmendel Sep 8, 2017
cfcb002
Merge branch 'master' of https://github.com/pandas-dev/pandas into ts…
jbrockmendel Sep 15, 2017
ff57861
Remove duplicate file
jbrockmendel Sep 15, 2017
62cc7b0
Reviewer comments; import try_parse_xyz directly from tslibs.parsing
jbrockmendel Sep 17, 2017
21046b3
flake8 fixup
jbrockmendel Sep 18, 2017
a5f8eca
Merge branch 'master' of https://github.com/pandas-dev/pandas into ts…
jbrockmendel Sep 23, 2017
364a6b0
keep _DATEUTIL_LEXER_SPLIT private to tslibs.parsing
jbrockmendel Sep 24, 2017
f89d11e
Merge branch 'master' of https://github.com/pandas-dev/pandas into ts…
jbrockmendel Sep 25, 2017
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
lint fixup
  • Loading branch information
@jbrockmendel
jbrockmendel committed Aug 30, 2017
commit c765e26c6fd30e07e987b404ec0d8a3e23d770f7
6 changes: 2 additions & 4 deletions pandas/core/tools/datetimes.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
from collections import MutableMapping

from pandas._libs import lib, tslib
from pandas._libs.tslib import (
from pandas._libs.tslib import ( # noqa
parse_time_string,
_format_is_iso,
Copy link
Contributor

Choose a reason for hiding this comment

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

can come back in future and de-privatize these in tslib.parsing (and change here)

Copy link
Member Author

Choose a reason for hiding this comment

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

Just added to TODO list in #17652

_DATEUTIL_LEXER_SPLIT,
Copy link
Contributor

Choose a reason for hiding this comment

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

I think _DATEUTIL_LEXER_SPLIT Is now private to tstlib.parsing.

Copy link
Member Author

Choose a reason for hiding this comment

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

Just pushed a commit that removes this import.

Expand All @@ -22,12 +22,10 @@
is_numeric_dtype)
from pandas.core.dtypes.generic import (
ABCIndexClass, ABCSeries,
ABCDataFrame, ABCDateOffset)
ABCDataFrame)
from pandas.core.dtypes.missing import notna
from pandas.core import algorithms

import pandas.compat as compat


def _infer_tzinfo(start, end):
def _infer(a, b):
Expand Down