pylantir 0.0.7__tar.gz → 0.0.9__tar.gz

pylantir-0.0.9/LICENSE

@@ -0,0 +1,21 @@
+    MIT License
+
+    Copyright (c) Microsoft Corporation.
+
+    Permission is hereby granted, free of charge, to any person obtaining a copy
+    of this software and associated documentation files (the "Software"), to deal
+    in the Software without restriction, including without limitation the rights
+    to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+    copies of the Software, and to permit persons to whom the Software is
+    furnished to do so, subject to the following conditions:
+
+    The above copyright notice and this permission notice shall be included in all
+    copies or substantial portions of the Software.
+
+    THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+    IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+    FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+    AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+    LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+    OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+    SOFTWARE

{pylantir-0.0.7 → pylantir-0.0.9}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pylantir
-Version: 0.0.7
+Version: 0.0.9
 Summary: Python - DICOM Modality WorkList
 Author-email: Milton Camacho <miltoncamachoicc@gmail.com>
 Requires-Python: >=3.11.1
@@ -13,12 +13,14 @@ Classifier: Programming Language :: Python :: 3.8
 Classifier: Programming Language :: Python :: 3.9
 Classifier: Programming Language :: Python :: 3.10
 Classifier: Programming Language :: Python :: 3.11
+License-File: LICENSE
 Requires-Dist: pynetdicom
 Requires-Dist: sqlalchemy
 Requires-Dist: PyCap
 Requires-Dist: uuid
 Requires-Dist: coloredlogs
 Requires-Dist: python-dotenv
+Requires-Dist: pandas
 Requires-Dist: pyspark>=3.0.0 ; extra == "spark"
 Requires-Dist: bandit[toml]==1.7.5 ; extra == "test"
 Requires-Dist: black==23.3.0 ; extra == "test"
@@ -29,7 +31,7 @@ Requires-Dist: flake8-formatter_junit_xml ; extra == "test"
 Requires-Dist: flake8 ; extra == "test"
 Requires-Dist: flake8-pyproject ; extra == "test"
 Requires-Dist: pre-commit==3.3.1 ; extra == "test"
-Requires-Dist: pylint==2.17.4 ; extra == "test"
+Requires-Dist: pylint==3.3.6 ; extra == "test"
 Requires-Dist: pylint_junit ; extra == "test"
 Requires-Dist: pytest-cov==4.0.0 ; extra == "test"
 Requires-Dist: pytest-mock<3.10.1 ; extra == "test"
@@ -75,7 +77,7 @@ export REDCAP_API_TOKEN=<your API token>
 Start a server called with AEtitle MWL_SERVER.
 
 ```bash
-pylantir start --ip 127.0.0.1 --port 4242 --AEtitle MWL_SERVER
+pylantir start --ip 127.0.0.1 --port 4242 --AEtitle MWL_SERVER --pylantir_config Path/to/your/config.json
 ```
 
 ## Tests
@@ -132,9 +134,8 @@ usage: pylantir [-h] [--AEtitle AETITLE] [--ip IP] [--port PORT] [--pylantir_con
 - **--port PORT**: Port for the server
 - **--pylantir_config PYLANTIR_CONFIG**: Path to the configuration JSON file containing pylantir configs:
   - **allowed_aet**: List of allowed AE titles e.g. `["MRI_SCANNER", "MRI_SCANNER_2"]`
-  - **mri_visit_session_mapping**: Mapping of MRI visit to session e.g., `{"T1": "1", "T2": "2"}`
   - **site**: Site ID:string
-  - **protocol**: `{"site": "protocol_name"}`
+  - **protocol**: `{"site": "protocol_name", "mapping": "HIS/RIS mapping"}`
   - **redcap2wl**: Dictionary of REDCap fields to worklist fields mapping e.g., `{"redcap_field": "worklist_field"}`
 - **--mpps_action {create,set}**: Action to perform for MPPS either create or set
 - **--mpps_status {COMPLETED,DISCONTINUED}**: Status to set for MPPS either COMPLETED or DISCONTINUED
@@ -152,20 +153,17 @@ As a default pylantir will try to read a JSON structured file with the following
   "db_echo": "False",
   "db_update_interval": 60,
   "allowed_aet": [],
-  "mri_visit_session_mapping": {
-    "t1_arm_1": "1",
-    "t2_arm_1": "2",
-    "t3_arm_1": "3"
-  },
   "site": "792",
   "redcap2wl": {
     "study_id": "study_id",
+    "instrument": "redcap_repeat_instrument",
+    "session_id" : "mri_instance",
     "family_id": "family_id",
     "youth_dob_y": "youth_dob_y",
     "t1_date": "t1_date",
     "demo_sex": "demo_sex",
-    "scheduled_date": "scheduled_start_date",
-    "scheduled_time": "scheduled_start_time",
+    "scheduled_date": "mri_date",
+    "scheduled_time": "mri_time",
     "mri_wt_lbs": "patient_weight_lb",
     "referring_physician": "referring_physician_name",
     "performing_physician": "performing_physician",
@@ -173,7 +171,8 @@ As a default pylantir will try to read a JSON structured file with the following
     "status": "performed_procedure_step_status"
   },
   "protocol": {
-    "792": "BRAIN_MRI_3T"
+    "792": "BRAIN_MRI_3T",
+    "mapping": "GEHC"
   }
 }
 ```

{pylantir-0.0.7 → pylantir-0.0.9}/README.md

@@ -30,7 +30,7 @@ export REDCAP_API_TOKEN=<your API token>
 Start a server called with AEtitle MWL_SERVER.
 
 ```bash
-pylantir start --ip 127.0.0.1 --port 4242 --AEtitle MWL_SERVER
+pylantir start --ip 127.0.0.1 --port 4242 --AEtitle MWL_SERVER --pylantir_config Path/to/your/config.json
 ```
 
 ## Tests
@@ -87,9 +87,8 @@ usage: pylantir [-h] [--AEtitle AETITLE] [--ip IP] [--port PORT] [--pylantir_con
 - **--port PORT**: Port for the server
 - **--pylantir_config PYLANTIR_CONFIG**: Path to the configuration JSON file containing pylantir configs:
   - **allowed_aet**: List of allowed AE titles e.g. `["MRI_SCANNER", "MRI_SCANNER_2"]`
-  - **mri_visit_session_mapping**: Mapping of MRI visit to session e.g., `{"T1": "1", "T2": "2"}`
   - **site**: Site ID:string
-  - **protocol**: `{"site": "protocol_name"}`
+  - **protocol**: `{"site": "protocol_name", "mapping": "HIS/RIS mapping"}`
   - **redcap2wl**: Dictionary of REDCap fields to worklist fields mapping e.g., `{"redcap_field": "worklist_field"}`
 - **--mpps_action {create,set}**: Action to perform for MPPS either create or set
 - **--mpps_status {COMPLETED,DISCONTINUED}**: Status to set for MPPS either COMPLETED or DISCONTINUED
@@ -107,20 +106,17 @@ As a default pylantir will try to read a JSON structured file with the following
   "db_echo": "False",
   "db_update_interval": 60,
   "allowed_aet": [],
-  "mri_visit_session_mapping": {
-    "t1_arm_1": "1",
-    "t2_arm_1": "2",
-    "t3_arm_1": "3"
-  },
   "site": "792",
   "redcap2wl": {
     "study_id": "study_id",
+    "instrument": "redcap_repeat_instrument",
+    "session_id" : "mri_instance",
     "family_id": "family_id",
     "youth_dob_y": "youth_dob_y",
     "t1_date": "t1_date",
     "demo_sex": "demo_sex",
-    "scheduled_date": "scheduled_start_date",
-    "scheduled_time": "scheduled_start_time",
+    "scheduled_date": "mri_date",
+    "scheduled_time": "mri_time",
     "mri_wt_lbs": "patient_weight_lb",
     "referring_physician": "referring_physician_name",
     "performing_physician": "performing_physician",
@@ -128,7 +124,8 @@ As a default pylantir will try to read a JSON structured file with the following
     "status": "performed_procedure_step_status"
   },
   "protocol": {
-    "792": "BRAIN_MRI_3T"
+    "792": "BRAIN_MRI_3T",
+    "mapping": "GEHC"
   }
 }
 ```

{pylantir-0.0.7 → pylantir-0.0.9}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "flit_core.buildapi"
 
 [project]
 name =  "pylantir"
-version = "0.0.7"
+version = "0.0.9"
 authors = [
     {name = "Milton Camacho", email = "miltoncamachoicc@gmail.com"},
 ]
@@ -30,6 +30,7 @@ dependencies = [
     "uuid",
     "coloredlogs",
     "python-dotenv",
+    "pandas",
 ]
 
 [project.scripts]
@@ -49,7 +50,7 @@ test = [
     "flake8",
     "flake8-pyproject",
     "pre-commit==3.3.1",
-    "pylint==2.17.4",
+    "pylint==3.3.6",
     "pylint_junit",
     "pytest-cov==4.0.0",
     "pytest-mock<3.10.1",

{pylantir-0.0.7 → pylantir-0.0.9}/src/pylantir/cli/run.py

@@ -212,9 +212,6 @@ def main() -> None:
         # Extract allowed AE Titles (default to empty list if missing)
         allowed_aet = config.get("allowed_aet", [])
 
-        # Extract mri_visit_session_mapping (default to empty list if missing)
-        mri_visit_session_mapping = config.get("mri_visit_session_mapping", {})
-
         # Extract the site id
         site = config.get("site", None)
 
@@ -229,7 +226,6 @@ def main() -> None:
         with ThreadPoolExecutor(max_workers=2) as executor:
             future = executor.submit(
                 sync_redcap_to_db_repeatedly,
-                mri_visit_mapping=mri_visit_session_mapping,
                 site_id=site,
                 protocol=protocol,
                 redcap2wl=redcap2wl,

{pylantir-0.0.7 → pylantir-0.0.9}/src/pylantir/models.py

@@ -37,6 +37,10 @@ class WorklistItem(Base):
     procedure_description = Column(String(200))
     protocol_name = Column(String(100))
     station_name = Column(String(100))
+    try:
+        hisris_coding_designator = Column(String(100))
+    except:
+        lgr.warning("Could not get hisris_coding_designator check models.py ")
     performed_procedure_step_status = Column(String, default="SCHEDULED")
 
 

{pylantir-0.0.7 → pylantir-0.0.9}/src/pylantir/mwl_server.py

@@ -94,10 +94,16 @@ def row_to_mwl_dataset(row: WorklistItem) -> Dataset:
     sps.ScheduledProcedureStepStatus = row.performed_procedure_step_status or "SCHEDULED"
 
     # Protocol Code Sequence
+    # you need to map action code (CodeValue) and coding scheme designator (CodingSchemeDesignator) for this to work
     if row.protocol_name:
         protocol_seq = Dataset()
-        protocol_seq.CodeValue = row.protocol_name[:16]
-        protocol_seq.CodingSchemeDesignator = "LOCAL"
+        protocol_seq.CodeValue = row.protocol_name
+        # protocol_seq.CodeValue = "CPIP"
+
+        # protocol_seq.ActionCode = "cpipmar03"
+        # protocol_seq.CodingSchemeDesignator = "GEHC"
+        protocol_seq.CodingSchemeDesignator = row.hisris_coding_designator
+        # protocol_seq.CodeMeaning = row.protocol_name
         protocol_seq.CodeMeaning = row.protocol_name
         sps.ScheduledProtocolCodeSequence = [protocol_seq]
 
@@ -149,34 +155,27 @@ def handle_mpps_n_create(event):
     """Handles N-CREATE for MPPS (Procedure Start)."""
     req = event.request
 
-    if req.AffectedSOPInstanceUID is None:
-        lgr.error("MPPS N-CREATE failed: Missing Affected SOP Instance UID")
-        return 0x0106, None  # Invalid Attribute Value
-
-    # Prevent duplicate MPPS instances
-    if req.AffectedSOPInstanceUID in managed_instances:
-        lgr.error("MPPS N-CREATE failed: Duplicate SOP Instance UID")
-        return 0x0111, None  # Duplicate SOP Instance
-
     attr_list = event.attribute_list
-
-    if "PerformedProcedureStepStatus" not in attr_list:
-        lgr.error("MPPS N-CREATE failed: Missing PerformedProcedureStepStatus")
-        return 0x0120, None  # Missing Attribute
-    if attr_list.PerformedProcedureStepStatus.upper() != "IN PROGRESS":
-        lgr.error("MPPS N-CREATE failed: Invalid PerformedProcedureStepStatus")
-        return 0x0106, None  # Invalid Attribute Value
-
     ds = Dataset()
     ds.SOPClassUID = ModalityPerformedProcedureStep
-    ds.SOPInstanceUID = req.AffectedSOPInstanceUID
-
-    # Copy attributes
+    ds.SOPInstanceUID = req.AffectedSOPInstanceUID or "UNKNOWN_UID"
     ds.update(attr_list)
 
     # Store MPPS instance
     managed_instances[ds.SOPInstanceUID] = ds
 
+    # Validation logic (log warnings, don't return errors to MRI scanner)
+    if not req.AffectedSOPInstanceUID:
+        lgr.warning("MPPS N-CREATE: Missing Affected SOP Instance UID")
+    elif req.AffectedSOPInstanceUID in managed_instances:
+        lgr.warning("MPPS N-CREATE: Duplicate SOP Instance UID received")
+
+    status = attr_list.get("PerformedProcedureStepStatus", "").upper()
+    if not status:
+        lgr.warning("MPPS N-CREATE: Missing PerformedProcedureStepStatus")
+    elif status != "IN PROGRESS":
+        lgr.warning(f"MPPS N-CREATE: Unexpected PerformedProcedureStepStatus = {status}")
+
     # Update database: Set status to IN_PROGRESS
     patient_id = ds.get("PatientID", None)
     session = Session()
@@ -187,32 +186,31 @@ def handle_mpps_n_create(event):
             session.commit()
             lgr.info(f"DB updated: PatientID {patient_id} set to IN_PROGRESS")
     else:
-        lgr.warning("MPPS N-SET received without StudyInstanceUID. Database update skipped.")
-    session.close()
-
-    lgr.info(f"MPPS N-CREATE success: {ds.SOPInstanceUID} set to IN PROGRESS")
+        lgr.warning("MPPS N-CREATE: No PatientID found in attributes. DB update skipped.")
 
-    return 0x0000, ds  # Success
+    session.close()
+    lgr.info(f"MPPS N-CREATE processed: {ds.SOPInstanceUID}")
 
+    return 0x0000, ds  # Always return Success
 
 def handle_mpps_n_set(event):
     """Handles N-SET for MPPS (Procedure Completion)."""
     req = event.request
-    if req.RequestedSOPInstanceUID not in managed_instances:
-        lgr.error("MPPS N-SET failed: SOP Instance not recognized")
-        return 0x0112, None  # No Such Object Instance
+    sop_uid = req.RequestedSOPInstanceUID
 
-    ds = managed_instances[req.RequestedSOPInstanceUID]
-    mod_list = event.attribute_list
+    if sop_uid not in managed_instances:
+        lgr.warning(f"MPPS N-SET: Unknown SOP Instance UID {sop_uid}")
+        ds = Dataset()
+        ds.SOPInstanceUID = sop_uid
+        return 0x0000, ds  # Still return success
 
-    # Update MPPS instance
+    ds = managed_instances[sop_uid]
+    mod_list = event.attribute_list
     ds.update(mod_list)
 
-    # Log status update
     new_status = ds.get("PerformedProcedureStepStatus", None)
     patient_id = ds.get("PatientID", None)
 
-    # Update database
     session = Session()
     if patient_id and new_status:
         entry = session.query(WorklistItem).filter_by(patient_id=patient_id).first()
@@ -225,11 +223,18 @@ def handle_mpps_n_set(event):
                 entry.performed_procedure_step_status = "DISCONTINUED"
                 session.commit()
                 lgr.info(f"DB updated: PatientID {patient_id} set to DISCONTINUED")
+            else:
+                lgr.warning(f"MPPS N-SET: Unrecognized status {new_status}")
+        else:
+            lgr.warning(f"MPPS N-SET: No DB entry found for PatientID {patient_id}")
+    else:
+        lgr.warning("MPPS N-SET: Missing PatientID or status. No DB update.")
+
     session.close()
+    lgr.info(f"MPPS N-SET processed: {sop_uid} -> {new_status}")
 
-    lgr.info(f"MPPS N-SET success: {req.RequestedSOPInstanceUID} updated to {new_status}")
+    return 0x0000, ds  # Always return Success
 
-    return 0x0000, ds  # Success
 
 
 # --------------------------------------------------------------------

{pylantir-0.0.7 → pylantir-0.0.9}/src/pylantir/redcap_to_db.py

@@ -1,5 +1,6 @@
 import os
 import logging
+import pandas as pd
 from redcap import Project
 import uuid
 from sqlalchemy.orm import sessionmaker
@@ -22,7 +23,7 @@ Session = sessionmaker(bind=engine)
 
 
 def fetch_redcap_entries(redcap_fields: list) -> list:
-    """Fetch scheduled procedures from REDCap using PyCap."""
+    """Fetch REDCap entries using PyCap and return a list of filtered dicts."""
     project = Project(REDCAP_API_URL, REDCAP_API_TOKEN)
 
     if not redcap_fields:
@@ -31,8 +32,6 @@ def fetch_redcap_entries(redcap_fields: list) -> list:
 
     # Fetch metadata to get valid REDCap field names
     valid_fields = {field["field_name"] for field in project.export_metadata()}
-
-    # Only keep fields that actually exist in REDCap
     redcap_fields = [field for field in redcap_fields if field in valid_fields]
 
     if not redcap_fields:
@@ -41,19 +40,43 @@ def fetch_redcap_entries(redcap_fields: list) -> list:
 
     lgr.info(f"Fetching REDCap data for fields: {redcap_fields}")
 
-    # Fetch records
-    records = project.export_records(fields=redcap_fields, format_type="json")
-
-    # Ensure all requested fields exist in every record (fill missing fields with None)
-    for record in records:
-        record["redcap_event_name"] = record.get("redcap_event_name", "UNKNOWN_EVENT")
-        for field in redcap_fields:
-            record.setdefault(field, None)  # Fill missing fields with None
+    # Export data
+    records = project.export_records(fields=redcap_fields, format_type="df")
 
-    if not records:
+    if records.empty:
         lgr.warning("No records retrieved from REDCap.")
+        return []
+
+    filtered_records = []
+
+    # Group by 'record_id' (index level 0)
+    for record_id, group in records.groupby(level=0):
+
+        # Try to get baseline (non-repeated instrument) values
+        baseline_rows = group[group['redcap_repeat_instrument'].isna()]
+        baseline_row = baseline_rows.iloc[0] if not baseline_rows.empty else {}
+
+        # Filter for valid MRI rows only
+        mri_rows = group[
+            (group["redcap_repeat_instrument"] == "mri") &
+            (group.get("mri_instance").notna()) &
+            (group.get("mri_instance") != "")
+        ]
+
+        for _, mri_row in mri_rows.iterrows():
+            record = {"record_id": record_id}
+
+            # Merge fields from baseline and mri_row, only include requested fields
+            for field in redcap_fields:
+                record[field] = (
+                    mri_row.get(field)
+                    if pd.notna(mri_row.get(field))
+                    else baseline_row.get(field)
+                )
+
+            filtered_records.append(record)
 
-    return records
+    return filtered_records
 
 # TODO: Implement age binning for paricipants
 def age_binning():
@@ -75,20 +98,7 @@ def convert_weight(weight, weight_unit):
     return weight, round(weight / 0.453592, 2)  # (kg, lb)
 
 
-def mapping_redcap_event_name_to_ses_id(mri_visit_mapping, redcap_event):
-    """Map REDCap event name to SES ID."""
-    try:
-        ses_id = mri_visit_mapping.get(redcap_event, None)
-        if ses_id is None:
-            raise ValueError(f"SES ID not found for REDCap event: {redcap_event}")
-        return ses_id
-    except Exception as e:
-        lgr.error(f"Error mapping REDCap event name to SES ID: {e}")
-        return None
-
-
 def sync_redcap_to_db(
-    mri_visit_mapping: dict,
     site_id: str,
     protocol: dict,
     redcap2wl: dict,
@@ -97,11 +107,13 @@ def sync_redcap_to_db(
 
     if not redcap2wl:
         lgr.error("No field mapping (redcap2wl) provided for syncing.")
+        return
 
     session = Session()
 
+    #TODO: Implement the repeat visit mapping
     # Extract the REDCap fields that need to be pulled
-    default_fields = ["study_id", "family_id", "youth_dob_y", "t1_date", "demo_sex"]
+    default_fields = ["record_id", "study_id", "redcap_repeat_instrument", "mri_instance", "mri_date", "mri_time", "family_id", "youth_dob_y", "t1_date", "demo_sex"]
     redcap_fields = list(redcap2wl.keys())
 
     # Ensure certain default fields are always present
@@ -116,12 +128,14 @@ def sync_redcap_to_db(
         study_id = study_id.split('-')[-1] if study_id else None
         family_id = record.get("family_id")
         family_id = family_id.split('-')[-1] if family_id else None
-        ses_id = mapping_redcap_event_name_to_ses_id(
-            mri_visit_mapping, record.get("redcap_event_name")
-        )
+        repeat_id = record.get("redcap_repeat_instance") if record.get("redcap_repeat_instance") != "" else "1" # Default to 1 if not set
+        lgr.debug(f"Processing record for Study ID: {study_id} and Family ID: {family_id}")
+        lgr.debug(f"This is the repeat event {repeat_id}")
+        ses_id = record.get("mri_instance")
 
         PatientName = f"cpip-id-{study_id}^fa-{family_id}"
-        PatientID = f"sub-{study_id}_ses-{ses_id}_fam-{family_id}_site-{site_id}"
+        PatientID = f"sub_{study_id}_ses_{ses_id}_fam_{family_id}_site_{site_id}"
+        PatientID_ = f"sub-{study_id}_ses-{ses_id}_fam-{family_id}_site-{site_id}"
 
         if not PatientID:
             lgr.warning("Skipping record due to missing Study ID.")
@@ -137,14 +151,25 @@ def sync_redcap_to_db(
             .first()
         )
 
+        existing_entry_ = (
+            session.query(WorklistItem)
+            .filter_by(patient_id=PatientID_)
+            .first()
+        )
         if existing_entry:
             logging.debug(f"Updating existing worklist entry for PatientID {PatientID}")
+        elif existing_entry_:
+            logging.debug(f"Updating existing worklist entry for PatientID {PatientID_}")
+            existing_entry = existing_entry_
+
+        if existing_entry:
             existing_entry.patient_name = PatientName
             existing_entry.patient_id = PatientID
             existing_entry.patient_birth_date = record.get("youth_dob_y", "19000101")
             existing_entry.patient_sex = record.get("demo_sex")
             existing_entry.modality = record.get("modality", "MR")
-
+            existing_entry.scheduled_start_date = record.get("mri_date")
+            existing_entry.scheduled_start_time = record.get("mri_time")
             # Dynamically update DICOM worklist fields from REDCap
             for redcap_field, dicom_field in redcap2wl.items():
                 if redcap_field in record:
@@ -169,9 +194,10 @@ def sync_redcap_to_db(
                 patient_birth_date=f"{record.get('youth_dob_y', '2012')}0101",
                 patient_sex=record.get("demo_sex"),
                 modality=record.get("modality", "MR"),
-                # scheduled_start_date=record.get("scheduled_date"),
-                # scheduled_start_time=record.get("scheduled_time"),
+                scheduled_start_date=record.get("mri_date"),
+                scheduled_start_time=record.get("mri_time"),
                 protocol_name=protocol.get(site_id, "DEFAULT_PROTOCOL"),
+                hisris_coding_designator=protocol.get("mapping", "scannermapper"),
                 # patient_weight_kg=patient_weight_kg,
                 patient_weight_lb=record.get("patient_weight_lb", ""),
                 # referring_physician_name=record.get("referring_physician"),
@@ -188,7 +214,6 @@ def sync_redcap_to_db(
 
 
 def sync_redcap_to_db_repeatedly(
-    mri_visit_mapping=None,
     site_id=None,
     protocol=None,
     redcap2wl=None,
@@ -201,7 +226,6 @@ def sync_redcap_to_db_repeatedly(
     while not STOP_EVENT.is_set():
         try:
             sync_redcap_to_db(
-                mri_visit_mapping=mri_visit_mapping,
                 site_id=site_id,
                 protocol=protocol,
                 redcap2wl=redcap2wl,
@@ -220,7 +244,6 @@ if __name__ == "__main__":
     # from a signal handler or from another part of your code.
     try:
         sync_redcap_to_db_repeatedly(
-            mri_visit_mapping=None,
             site_id=None,
             protocol=None,
             redcap2wl=None,