Skip to content

Commit 8d62b33

Browse files
Merge branch 'feat/format-code' of https://github.com/richardkoehler/App_ClinicalDBSAnnot into feat/format-code
2 parents 2aa28c3 + daba998 commit 8d62b33

2 files changed

Lines changed: 48 additions & 26 deletions

File tree

src/clinical_dbs_annotator/utils/longitudinal_exporter.py

Lines changed: 22 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -685,8 +685,16 @@ def _add_longitudinal_data_table(
685685
]
686686
display_cols = [c for c in lateral_df.columns if c not in columns_to_exclude]
687687

688-
lateral_cols = ["date", "laterality", "frequency", "anode", "cathode", "amplitude", "pulse_width"]
689-
common_cols = ["program_ID", "scale_name", "scale_value", "notes"]
688+
lateral_cols = [
689+
"date",
690+
"laterality",
691+
"frequency",
692+
"anode",
693+
"cathode",
694+
"amplitude",
695+
"pulse_width",
696+
]
697+
common_cols = ["group_ID", "scale_name", "scale_value", "notes"]
690698

691699
lateral_cols = [c for c in lateral_cols if c in display_cols]
692700
common_cols = [c for c in common_cols if c in display_cols]
@@ -706,10 +714,16 @@ def _add_longitudinal_data_table(
706714
section.page_width - section.left_margin - section.right_margin
707715
) / 914400
708716
base_w = {
709-
"date": 0.65, "laterality": 0.25, "program_ID": 0.35,
710-
"frequency": 0.45, "anode": 0.45, "cathode": 0.60,
711-
"amplitude": 0.60, "pulse_width": 0.50,
712-
"scale_name": 1.00, "scale_value": 0.55,
717+
"date": 0.65,
718+
"laterality": 0.25,
719+
"group_ID": 0.35,
720+
"frequency": 0.45,
721+
"anode": 0.45,
722+
"cathode": 0.60,
723+
"amplitude": 0.60,
724+
"pulse_width": 0.50,
725+
"scale_name": 1.00,
726+
"scale_value": 0.55,
713727
}
714728
widths = [base_w.get(c, 0.5) for c in ordered]
715729
if "notes" in ordered:
@@ -1368,7 +1382,8 @@ def _column_header(col: str) -> str:
13681382
"cathode": "-",
13691383
"amplitude": PLACEHOLDERS.get("amplitude", "Amp"),
13701384
"pulse_width": PLACEHOLDERS.get("pulse_width", "PW"),
1371-
"program_ID": "Prog", "laterality": "",
1385+
"group_ID": "Grp",
1386+
"laterality": "",
13721387
}
13731388
return m.get(col, col.replace("_", " ").title())
13741389

src/clinical_dbs_annotator/utils/session_exporter.py

Lines changed: 26 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -501,15 +501,15 @@ def _create_lateral_table_data(self, df):
501501
right_row["block_id"] = block_id
502502

503503
# Common columns (non-lateral) - use combined scales with internal lines
504-
left_row['program_ID'] = first_row.get('program_ID', '')
505-
left_row['scale_name'] = combined_scale_name
506-
left_row['scale_value'] = combined_scale_value
507-
left_row['notes'] = first_row.get('notes', '')
504+
left_row["group_ID"] = first_row.get("group_ID", "")
505+
left_row["scale_name"] = combined_scale_name
506+
left_row["scale_value"] = combined_scale_value
507+
left_row["notes"] = first_row.get("notes", "")
508508

509-
right_row['program_ID'] = first_row.get('program_ID', '')
510-
right_row['scale_name'] = combined_scale_name
511-
right_row['scale_value'] = combined_scale_value
512-
right_row['notes'] = first_row.get('notes', '')
509+
right_row["group_ID"] = first_row.get("group_ID", "")
510+
right_row["scale_name"] = combined_scale_name
511+
right_row["scale_value"] = combined_scale_value
512+
right_row["notes"] = first_row.get("notes", "")
513513

514514
# Lateral columns - map to generic names
515515
lateral_mappings = {
@@ -570,8 +570,15 @@ def _add_session_data_table(self, doc: Document, df_table: pd.DataFrame) -> None
570570
col for col in lateral_df.columns if col not in columns_to_exclude
571571
]
572572

573-
lateral_cols = ['laterality', 'frequency', 'anode', 'cathode', 'amplitude', 'pulse_width']
574-
common_cols = ['program_ID', 'scale_name', 'scale_value', 'notes']
573+
lateral_cols = [
574+
"laterality",
575+
"frequency",
576+
"anode",
577+
"cathode",
578+
"amplitude",
579+
"pulse_width",
580+
]
581+
common_cols = ["group_ID", "scale_name", "scale_value", "notes"]
575582

576583
lateral_cols = [col for col in lateral_cols if col in display_columns]
577584
common_cols = [col for col in common_cols if col in display_columns]
@@ -588,15 +595,15 @@ def _add_session_data_table(self, doc: Document, df_table: pd.DataFrame) -> None
588595
) / 914400
589596

590597
base_in = {
591-
'laterality': 0.30,
592-
'program_ID': 0.40,
593-
'frequency': 0.50,
594-
'anode': 0.45,
595-
'cathode': 0.60,
596-
'amplitude': 0.60,
597-
'pulse_width': 0.50,
598-
'scale_name': 1.10,
599-
'scale_value': 0.60,
598+
"laterality": 0.30,
599+
"group_ID": 0.40,
600+
"frequency": 0.50,
601+
"anode": 0.45,
602+
"cathode": 0.60,
603+
"amplitude": 0.60,
604+
"pulse_width": 0.50,
605+
"scale_name": 1.10,
606+
"scale_value": 0.60,
600607
}
601608
widths_in = [base_in.get(c, 0.5) for c in ordered_columns]
602609
if "notes" in ordered_columns:

0 commit comments

Comments
 (0)