TYP: simplify read_csv/fwf/table overloads #57476
Conversation
| skip_blank_lines: bool = True, | ||
| # Datetime Handling | ||
| parse_dates: bool | Sequence[Hashable] = False, | ||
| parse_dates: bool | Sequence[Hashable] | None = None, |
There was a problem hiding this comment.
Not sure what to do here: None of the three methods document None but it is the default in read_csv.
A few options:
- Document it, but keep it
Falseinread_table(and add a deprecation warning, that it will be None in the future) - Document it, change it to
Noneinread_table(might need a deprecationwarnign first? edit: could probably directly change it with 3.0 but add a whatsnew entry?) - Disallow
None, add a deprecation warning in the other methods
There was a problem hiding this comment.
Document it, change it to None in read_table (might need a deprecationwarnign first? edit: could probably directly change it with 3.0 but add a whatsnew entry?)
I would support this and mark it as an Other API changes
There was a problem hiding this comment.
Okay, the current commits already do that :)
| "infer_nrows": infer_nrows, | ||
| "engine": "python-fwf", | ||
| "iterator": iterator, | ||
| "chunksize": chunksize, |
There was a problem hiding this comment.
mypy doesn't like assigning new fields to a TypedDict
| widths: None | ||
| | ||
| | ||
| class _read_shared(TypedDict, total=False): |
There was a problem hiding this comment.
Not sure whether this should move to _typing?
| infer_nrows : int, default 100 | ||
| The number of rows to consider when letting the parser determine the | ||
| `colspecs`. | ||
| dtype_backend : {'numpy_nullable', 'pyarrow'}, default 'numpy_nullable' |
There was a problem hiding this comment.
Did this get moved somewhere else?
There was a problem hiding this comment.
I moved it into the kwds to share it among the read-functions. The doc-test was failing as it did not find the corresponding keyword in the function signature anymore.
| Thanks @twoertwein |
* TYP: simplify read_csv/fwf/table overloads * isort (keep thinking ruff is configured to do that too) * another one * fix docs? * sort whatsnew entry
2 participants
No description provided.