Add a delimiters parameter to the csv.Sniffer constructor
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- Python
- Star
- 77.2k
- Fork
- 35.9k
- Chỉ số merge pull request
- Chỉ số pull request đang chờ
Mô tả
Feature or enhancement
csv.Sniffer.sniff() accepts an optional delimiters argument which restricts the set of characters that can be guessed as a delimiter. has_header() has no such argument, so the restriction cannot be applied to it.
I propose to add an optional delimiters parameter to the constructor, so that it applies to the sniffer rather than to a single call:
sniffer = csv.Sniffer(',;\t')
if sniffer.has_header(sample):
...
dialect = sniffer.sniff(sample)
It is used by both sniff() and has_header(). The delimiters argument of sniff(), if given, still takes precedence.
The given delimiters are also preferred in that order if several of them split the sample equally well: csv.Sniffer(';,') guesses ';' where csv.Sniffer(',;') guesses ','.
Linked PRs
- gh-155060
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Hướng nghiên cứu
Bắt đầu với các điểm vào csv.Sniffer, sniff() và has_header() được mô tả trong issue, sau đó kiểm tra các bài kiểm thử CSV hiện có để biết hành vi hiện tại. Hoàn thành nghĩa là giới hạn delimiters của constructor được cả hai method sử dụng, sniff() có thể ghi đè giới hạn đó và các candidate có kết quả bằng nhau tuân theo thứ tự delimiter được cung cấp.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- python
- Lĩnh vực
- data
- Loại issue
- Tính năng
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức phù hợp với người mới
- 35/100