Merge pull request #325 from cta-observatory/issue322

Changes about the calibration prod_id - Issue #322

Author: marialainez

src/osa/scripts/calibration_pipeline.py

@@ -20,6 +20,7 @@
 from osa.utils.cliopts import calibration_pipeline_cliparsing
 from osa.utils.logging import myLogger
 from osa.workflow.stages import DRS4PedestalStage, ChargeCalibrationStage
+from osa.utils.utils import get_lstchain_version
 
 __all__ = [
     "calibration_sequence",
@@ -46,25 +47,30 @@ def drs4_pedestal_command(drs4_pedestal_run_id: int) -> list:
     base_dir = Path(cfg.get("LST1", "BASE")).resolve()
     r0_dir = Path(cfg.get("LST1", "R0_DIR")).resolve()
     command = cfg.get("lstchain", "drs4_baseline")
+    prod_id = get_lstchain_version()
     return [
         command,
         "-r", str(drs4_pedestal_run_id),
         "-b", base_dir,
         f"--r0-dir={r0_dir}",
         "--no-progress",
+        "-v",prod_id,
     ]
 
 def calibration_file_command(drs4_pedestal_run_id: int, pedcal_run_id: int) -> list:
     """Build the create_calibration_file command."""
     base_dir = Path(cfg.get("LST1", "BASE")).resolve()
     r0_dir = Path(cfg.get("LST1", "R0_DIR")).resolve()
     command = cfg.get("lstchain", "charge_calibration")
+    prod_id = get_lstchain_version()
     cmd = [
         command,
         "-p", str(drs4_pedestal_run_id),
         "-r", str(pedcal_run_id),
         "-b", base_dir,
         f"--r0-dir={r0_dir}",
+        "-v",prod_id,
+
     ]
     # In case of problems with trigger tagging:
     if cfg.getboolean("lstchain", "use_ff_heuristic_id"):

src/osa/scripts/tests/test_osa_scripts.py

@@ -349,6 +349,8 @@ def test_drs4_pedestal_cmd(base_test_dir):
         base_test_dir,
         f"--r0-dir={r0_dir}",
         "--no-progress",
+        "-v","v0.10.18"
+
     ]
     assert cmd == expected_command
 
@@ -367,6 +369,7 @@ def test_calibration_file_cmd(base_test_dir):
         "-b",
         base_test_dir,
         f"--r0-dir={r0_dir}",
+        "-v","v0.10.18"
     ]
     assert cmd == expected_command