Record.wrsamp() fails with a TypeError when the record has no d_signal
Maintainer thường phản hồi trong vòng 1 ngày
Đánh giá
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 70/100
Hướng nghiên cứu
Xác định lớp Record (có thể nằm trong wfdb/record.py) và kiểm tra wrsamp() và calc_checksum(). Chạy đoạn mã được cung cấp để tái hiện TypeError. Quyết định giữa việc chuyển đổi p_signal thành d_signal hoặc ném ra một lỗi rõ ràng; done nghĩa là việc ghi bản ghi thành công hoặc thất bại với một thông báo hữu ích.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Record.wrsamp() fails with an unhelpful error for a record read with the default physical=True:
import wfdb
record = wfdb.rdrecord("sample-data/100", sampto=1000)
record.wrsamp(write_dir="out")
# TypeError: unsupported operand type(s) for %: 'NoneType' and 'int'
The record only has p_signal, so d_signal is None. Record.wrsamp() first updates the checksums with calc_checksum(), which computes np.sum(self.d_signal, 0) % 65536 and fails on None. Reading with physical=False, or calling record.d_signal = record.adc() first, works.
#543 ran into the same error. Two possible fixes:
- Convert
p_signaltod_signalwith the record'sadc_gainandbaselinewhend_signalis missing, so that a record read withrdrecordcan be written back directly (which would also help with #500). - At least raise a clear error saying that
d_signalis required, and how to get it.
- Ngôn ngữ chính
- Jupyter Notebook
- Star
- 856
- Fork
- 326
- Merge trung bình
- 22 giờ 46 phút
- Pull request đã merge (30 ngày)
- 40
Chuẩn bị môi trường
Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.
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.
Issue khác của MIT-LCP/wfdb-python
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 72/100
MIT-LCP/wfdb-python#590 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 45/100
MIT-LCP/wfdb-python#568 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Add validation of segments when writing a `MultiRecord`Có thể đã có người làm Có pull request liên kết đang mở hoặc đã được merge. Đang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 48/100
MIT-LCP/wfdb-python#557 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 58/100
MIT-LCP/wfdb-python#554 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
WFDB path ignoredĐang mở
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
MIT-LCP/wfdb-python#545 ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của MIT-LCP/wfdb-python
Issue tương tự
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
ansys/pydpf-core#3620 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
correction metadata
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 72/100
acl-org/acl-anthology#10767 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
area:backend bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 77/100
scipp/esslivedata#1437 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
Maintainer thường phản hồi trong vòng 1 ngày
-
HandyModRTransform.deriv3 is wrong for m > 2Có thể đã có người làm @SajalDevX đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100