Skip to content
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

Suppress pandas FutureWarnings #165

Merged
merged 12 commits into from
Dec 2, 2024
Merged

Suppress pandas FutureWarnings #165

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
a8d8bc1
Suppress pandas FutureWarnings
JPhillips92 Nov 16, 2023
c255908
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] Nov 16, 2023
f313df1
Fix deprecated applymap usage
JPhillips92 Nov 20, 2023
a05c166
Merge branch 'master' into master
yehoshuadimarsky Feb 25, 2024
25ac07c
Merge remote-tracking branch 'yehoshuadimarsky/master'
JPhillips92 Jul 16, 2024
59647a0
Config fix
JPhillips92 Jul 16, 2024
ac491e8
Update poetry lock file
JPhillips92 Jul 16, 2024
bc953a8
Merge branch 'master' into master
JPhillips92 Sep 2, 2024
7c1812c
Merge branch 'master' into master
JPhillips92 Sep 5, 2024
113dd9f
Merge branch 'master' into master
JPhillips92 Nov 25, 2024
53754f9
Merge branch 'upstream'
JPhillips92 Dec 2, 2024
6971b28
Update usage of applymap in utils tests
JPhillips92 Dec 2, 2024
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
4 changes: 2 additions & 2 deletions bcpandas/constants.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -65,13 +65,13 @@ class BCPandasValueError(BCPandasException):

def get_delimiter(df: pd.DataFrame) -> str:
for delim in _DELIMITER_OPTIONS:
if not df.applymap(lambda x: delim in x if isinstance(x, str) else False).any().any():
if not df.map(lambda x: delim in x if isinstance(x, str) else False).any().any():
return delim
raise BCPandasValueError(error_msg.format(typ="delimiter", opts=_DELIMITER_OPTIONS))


def get_quotechar(df: pd.DataFrame) -> str:
for qc in _QUOTECHAR_OPTIONS:
if not df.applymap(lambda x: qc in x if isinstance(x, str) else False).any().any():
if not df.map(lambda x: qc in x if isinstance(x, str) else False).any().any():
return qc
raise BCPandasValueError(error_msg.format(typ="quote", opts=_QUOTECHAR_OPTIONS))
4 changes: 2 additions & 2 deletions tests/utils.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -53,14 +53,14 @@

def not_has_all_delims(df: pd.DataFrame) -> bool:
return not all(
df.applymap(lambda x: delim in x if isinstance(x, str) else False).any().any()
df.map(lambda x: delim in x if isinstance(x, str) else False).any().any()
for delim in _DELIMITER_OPTIONS
)


def not_has_all_quotechars(df: pd.DataFrame) -> bool:
return not all(
df.applymap(lambda x: qc in x if isinstance(x, str) else False).any().any()
df.map(lambda x: qc in x if isinstance(x, str) else False).any().any()
for qc in _QUOTECHAR_OPTIONS
)

Expand Down