Fix linter errors (ruff)

- Remove trailing whitespace from blank lines
- Split long lines to comply with 99 character limit
- Format code with ruff formatter for consistency

Co-authored-by: tsalo <8228902+tsalo@users.noreply.github.com>

Author: Copilot

xcp_d/ingression/utils.py

@@ -90,23 +90,25 @@ def write_scans_tsv(copy_dictionary, subject_dir_bids, subses_ents, out_dir=None
     scans_df = pd.DataFrame(list(scans_dict.items()), columns=['filename', 'source_file'])
     scans_tsv = os.path.join(subject_dir_bids, f'{subses_ents}_scans.tsv')
     scans_df.to_csv(scans_tsv, sep='\t', index=False)
-    
+
     # Also copy the scans.tsv to sourcedata/bids_conversion if out_dir is provided
     if out_dir is not None:
         # Parse subses_ents to extract subject and optional session
         # Format can be: "sub-01" or "sub-01_ses-01"
         parts = subses_ents.split('_')
         sub_part = parts[0]  # e.g., "sub-01"
-        
+
         # Build the sourcedata path with subject/session structure
         if len(parts) > 1 and parts[1].startswith('ses-'):
             # Has session: bids_conversion/sub-01/ses-01/
             ses_part = parts[1]  # e.g., "ses-01"
-            sourcedata_dir = os.path.join(out_dir, 'sourcedata', 'bids_conversion', sub_part, ses_part)
+            sourcedata_dir = os.path.join(
+                out_dir, 'sourcedata', 'bids_conversion', sub_part, ses_part
+            )
         else:
             # No session: bids_conversion/sub-01/
             sourcedata_dir = os.path.join(out_dir, 'sourcedata', 'bids_conversion', sub_part)
-        
+
         os.makedirs(sourcedata_dir, exist_ok=True)
         sourcedata_scans_tsv = os.path.join(sourcedata_dir, f'{subses_ents}_scans.tsv')
         scans_df.to_csv(sourcedata_scans_tsv, sep='\t', index=False)

xcp_d/tests/test_ingression_utils.py

@@ -409,15 +409,17 @@ def test_write_scans_tsv_with_out_dir(tmp_path):
         '/source/file2.nii.gz': [str(subject_dir / 'func' / 'sub-01_bold.nii.gz')],
     }
     ingress_utils.write_scans_tsv(copy_dict, str(subject_dir), 'sub-01', out_dir=str(out_dir))
-    
+
     # Check that scans.tsv exists in subject directory
     subject_scans_tsv = subject_dir / 'sub-01_scans.tsv'
     assert subject_scans_tsv.exists()
-    
+
     # Check that scans.tsv is also copied to sourcedata/bids_conversion/sub-01/
-    sourcedata_scans_tsv = out_dir / 'sourcedata' / 'bids_conversion' / 'sub-01' / 'sub-01_scans.tsv'
+    sourcedata_scans_tsv = (
+        out_dir / 'sourcedata' / 'bids_conversion' / 'sub-01' / 'sub-01_scans.tsv'
+    )
     assert sourcedata_scans_tsv.exists()
-    
+
     # Verify both files have the same content
     df_subject = pd.read_csv(subject_scans_tsv, sep='\t')
     df_sourcedata = pd.read_csv(sourcedata_scans_tsv, sep='\t')
@@ -434,16 +436,25 @@ def test_write_scans_tsv_with_session(tmp_path):
     copy_dict = {
         '/source/bold.nii.gz': [str(subject_dir / 'func' / 'sub-01_ses-01_bold.nii.gz')],
     }
-    ingress_utils.write_scans_tsv(copy_dict, str(subject_dir), 'sub-01_ses-01', out_dir=str(out_dir))
-    
+    ingress_utils.write_scans_tsv(
+        copy_dict, str(subject_dir), 'sub-01_ses-01', out_dir=str(out_dir)
+    )
+
     # Check both locations
     subject_scans_tsv = subject_dir / 'sub-01_ses-01_scans.tsv'
     assert subject_scans_tsv.exists()
-    
+
     # Check sourcedata/bids_conversion/sub-01/ses-01/
-    sourcedata_scans_tsv = out_dir / 'sourcedata' / 'bids_conversion' / 'sub-01' / 'ses-01' / 'sub-01_ses-01_scans.tsv'
+    sourcedata_scans_tsv = (
+        out_dir
+        / 'sourcedata'
+        / 'bids_conversion'
+        / 'sub-01'
+        / 'ses-01'
+        / 'sub-01_ses-01_scans.tsv'
+    )
     assert sourcedata_scans_tsv.exists()
-    
+
     df = pd.read_csv(sourcedata_scans_tsv, sep='\t')
     assert len(df) == 1
     assert 'sub-01_ses-01_bold.nii.gz' in df['filename'].values[0]