🐛 First two TRs not affected by bandpass filter

[shnizzedy]

Reported by Shinwon Park

Describe the bug

Two TRs are excluded from bandpass filters when 1D files have 3-row headers.

C-PAC/CPAC/nuisance/bandpass.py

Line 120 in 96db8b0

if line.startswith('#') or isinstance(line[0], str):

evaluates to True for any lines when they're read in as strings, so

C-PAC/CPAC/nuisance/bandpass.py

Lines 118 to 121 in 96db8b0

	for i in range(5):
	line = f.readline()
	if line.startswith('#') or isinstance(line[0], str):
	header.append(line)

is grabbing the first 5 lines as header regardless of their contents.

Preconfig

• default
• abcd-options
• anat-only
• blank
• ccs-options
• fmriprep-options
• fx-options
• monkey
• monkey-ABCD
• ndmg
• nhp-macaque
• preproc
• rbc-options
• rodent

Custom pipeline configuration

Anything with bandpass regressors

Expected behavior

All TRs are included in bandpass filter

Acceptance criteria

• All TRs are included in bandpass filter

C-PAC version

v1.8.6, v1.8.7

Container platform

Docker, Singularity