Skip to content

Raise error in read_csv when arguments header and prefix both are not None #31383

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 17 commits into from
Feb 3, 2020
Merged

Raise error in read_csv when arguments header and prefix both are not None #31383

Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
c053a8f
add error msg and test
rushabh-v Jan 28, 2020
071ba5b
run black pandas
rushabh-v Jan 28, 2020
ed8278e
remove extra quotes
rushabh-v Jan 28, 2020
7231f52
change location of the test
rushabh-v Jan 28, 2020
a5fb107
add whatsnew
rushabh-v Jan 28, 2020
13d8644
add gh
rushabh-v Jan 28, 2020
a003c51
change position of the raise
rushabh-v Jan 31, 2020
77c8ee0
change the location of the test
rushabh-v Feb 1, 2020
39cd6d5
sort imports
rushabh-v Feb 1, 2020
a2cb6a4
use isort
rushabh-v Feb 1, 2020
204dbc2
add the check in ParserBase
rushabh-v Feb 2, 2020
1bc350b
add a None constraint
rushabh-v Feb 2, 2020
a03d5b5
remove import read_csv
rushabh-v Feb 2, 2020
cc1f6f8
make requested changes
rushabh-v Feb 2, 2020
5fc40bd
Merge branch 'master' of https://github.com/pandas-dev/pandas into pr…
rushabh-v Feb 2, 2020
d962941
make requested changes
rushabh-v Feb 3, 2020
1f35f6c
Merge branch 'master' into pre-head
rushabh-v Feb 3, 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
4 changes: 4 additions & 0 deletions pandas/io/parsers.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1908,6 +1908,10 @@ def __init__(self, src, **kwds):
]
else:
self.names = list(range(self._reader.table_width))
elif self.prefix:
raise ValueError(
"Argument prefix must be None if argument header is not None"
)

# gh-9755
#
Expand Down
8 changes: 8 additions & 0 deletions pandas/tests/io/parser/test_header.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,14 @@ def test_read_with_bad_header(all_parsers):
parser.read_csv(s, header=[10])


def test_read_raises_on_header_prefix(all_parsers):
parser = all_parsers
msg = "Argument prefix must be None if argument header is not None"
with pytest.raises(ValueError, match=msg):
s = StringIO("0,1\n2,3")
parser.read_csv(s, header=0, prefix="_X")


def test_negative_header(all_parsers):
# see gh-27779
parser = all_parsers
Expand Down