Fix datasource parsing for on-off-bits arguments

gstlal/python/datasource.py

@@ -364,10 +364,10 @@ class DataSourceInfo:
 		shared_memory_part = parse_list_to_dict(options.shared_memory_partition)
 		state_channel_name = parse_list_to_dict(options.state_channel_name)
 		dq_channel_name = parse_list_to_dict(options.dq_channel_name)
-		state_vector_on_bits = parse_list_to_dict(options.state_vector_on_bits)
-		state_vector_off_bits = parse_list_to_dict(options.state_vector_off_bits)
-		dq_vector_on_bits = parse_list_to_dict(options.dq_vector_on_bits)
-		dq_vector_off_bits = parse_list_to_dict(options.dq_vector_off_bits)
+		state_vector_on_bits = parse_list_to_dict(options.state_vector_on_bits, value_transform=parse_int)
+		state_vector_off_bits = parse_list_to_dict(options.state_vector_off_bits, value_transform=parse_int)
+		dq_vector_on_bits = parse_list_to_dict(options.dq_vector_on_bits, value_transform=parse_int)
+		dq_vector_off_bits = parse_list_to_dict(options.dq_vector_off_bits, value_transform=parse_int)
 
 		return DataSourceInfo(data_source=options.data_source,
 							  block_size=options.block_size,

gstlal/tests/tests_pytest/test_datasource.py

@@ -502,6 +502,32 @@ class TestDataSourceInfoBasicSourceOptParse:
 			info = datasource.DataSourceInfo.from_optparse(options)
 			assert_info_mkbasicsrc(info, Detector.H1.value)
 
+	def test_on_off_bits_coercion(self):
+		"""Test that on off bits are being coerced"""
+		with GstLALTestManager() as gsttm:
+			args = [
+				'--data-source=white',
+				'--frame-cache={}'.format(gsttm.cache_path),
+				'--channel-name=H1=CHANNEL',
+				'--gps-start-time=1234567',
+				'--gps-end-time=1234568',
+				'--state-vector-on-bits=H1=3',
+				'--state-vector-on-bits=L1=3',
+				'--state-vector-on-bits=V1=1027',
+				'--state-vector-off-bits=H1=0',
+				'--state-vector-off-bits=L1=0',
+				'--state-vector-off-bits=V1=0'
+			]
+			parser = optparse.OptionParser(description="test legacy on off coercion")
+			datasource.append_options(parser)
+			options, filenames = parser.parse_args(args)
+
+			info = datasource.DataSourceInfo.from_optparse(options)
+			assert_info_mkbasicsrc(info, Detector.H1.value)
+			for k in ('H1', 'H2', 'V1', 'L1'):
+				for v in info.state_vector_on_off_bits[k]:
+					assert isinstance(v, int)
+
 
 @pytest.mark.requires_gstlal_ugly
 class TestDataSourceInfoBasicSourceOptParseUgly: