MIT-LCP / wfdb-python

Native Python WFDB package
MIT License
730 stars 298 forks source link

Add a test to check if wrsamp can write more than 8 channels to format 516 #451

Closed alistairewj closed 1 year ago

alistairewj commented 1 year ago

We should be able to read and write data with more than 8 channels to formats 508, 516, and 524. This test verifies that the digital signal generated when writing to format 516 is identical to the original signal loaded in from a sample record in format 16.

Should catch bugs like those fixed in #450

tompollard commented 1 year ago

@alistairewj more tests are always good! looks like the style checks failed. please could you check format and update the PR?

alistairewj commented 1 year ago

Ah OK I had autopep8 running and not black. Updated now, fingers crossed!