remove @metrics_logger decorators

spacetelescope · Sep 9, 2024 · 0d0e4da · 0d0e4da
1 parent 7b8c09f
commit 0d0e4da
Show file tree

Hide file tree

Showing 10 changed files with 0 additions and 25 deletions.
diff --git a/romancal/regtest/test_catalog.py b/romancal/regtest/test_catalog.py
@@ -2,7 +2,6 @@
 
 import asdf
 import pytest
-from metrics_logger.decorators import metrics_logger
 
 from romancal.source_catalog.source_catalog_step import SourceCatalogStep
 from romancal.stpipe import RomanStep
@@ -48,7 +47,6 @@ def check_catalog_fields(model, log, modeltype):
 
 @pytest.mark.bigdata
 @pytest.mark.soctests
-@metrics_logger("DMS374", "DMS375", "DMS376", "DMS386", "DMS387")
 def test_catalog_l3(rtdata, ignore_asdf_paths):
  # DMS374: positions on ICRF
  # DMS375: fluxes
@@ -76,7 +74,6 @@ def test_catalog_l3(rtdata, ignore_asdf_paths):
 
 @pytest.mark.bigdata
 @pytest.mark.soctests
-@metrics_logger("DMS374", "DMS375", "DMS376", "DMS386", "DMS387")
 def test_catalog_l2(rtdata, ignore_asdf_paths):
  # DMS374: positions on ICRF
  # DMS375: fluxes

diff --git a/romancal/regtest/test_mos_pipeline.py b/romancal/regtest/test_mos_pipeline.py
@@ -4,7 +4,6 @@
 
 import pytest
 import roman_datamodels as rdm
-from metrics_logger.decorators import metrics_logger
 
 from romancal.pipeline.mosaic_pipeline import MosaicPipeline
 
@@ -20,7 +19,6 @@ def passfail(bool_expr):
 
 @pytest.mark.bigdata
 @pytest.mark.soctests
-@metrics_logger("DMS356", "DMS374", "DMS400")
 def test_level3_mos_pipeline(rtdata, ignore_asdf_paths):
  """Tests for level 3 processing requirements DMS356"""
  rtdata.get_asn("WFI/image/L3_regtest_asn.json")
@@ -131,7 +129,6 @@ def test_level3_mos_pipeline(rtdata, ignore_asdf_paths):
 
 @pytest.mark.bigdata
 @pytest.mark.soctests
-@metrics_logger("DMS373")
 def test_hlp_mosaic_pipeline(rtdata, ignore_asdf_paths):
  """Tests for level 3 mosaic requirements DMS373"""
  rtdata.get_asn("WFI/image/L3_mosaic_asn.json")

diff --git a/romancal/regtest/test_ramp_fitting.py b/romancal/regtest/test_ramp_fitting.py
@@ -10,7 +10,6 @@
 
 import pytest
 import roman_datamodels as rdm
-from metrics_logger.decorators import metrics_logger
 
 from romancal.lib.dms import log_result
 from romancal.lib.suffix import replace_suffix
@@ -188,7 +187,6 @@ def test_rampfit_step(rampfit_result, rtdata_module, ignore_asdf_paths):
  except AssertionError as e:
  error_msgs.append(str(e))
 
- @metrics_logger(requirement)
  def test_success():
  assert not len(error_msgs), "\n".join(error_msgs)
 

diff --git a/romancal/regtest/test_resample.py b/romancal/regtest/test_resample.py
@@ -1,6 +1,5 @@
 import numpy as np
 import pytest
-from metrics_logger.decorators import metrics_logger
 from roman_datamodels import datamodels as rdm
 
 from romancal.resample.resample_step import ResampleStep
@@ -9,7 +8,6 @@
 from .regtestdata import compare_asdf
 
 
-@metrics_logger("DMS342", "DMS343", "DMS344", "DMS345")
 @pytest.mark.bigdata
 def test_resample_single_file(rtdata, ignore_asdf_paths):
  output_data = "mosaic_resamplestep.asdf"

diff --git a/romancal/regtest/test_tweakreg.py b/romancal/regtest/test_tweakreg.py
@@ -2,7 +2,6 @@
 import numpy as np
 import pytest
 from astropy import units as u
-from metrics_logger.decorators import metrics_logger
 from roman_datamodels import datamodels as rdm
 
 from romancal.stpipe import RomanStep
@@ -11,7 +10,6 @@
 from .regtestdata import compare_asdf
 
 
-@metrics_logger("DMS280", "DMS405", "DMS488")
 @pytest.mark.bigdata
 def test_tweakreg(rtdata, ignore_asdf_paths, tmp_path):
  # N.B.: uncal file is from simulator

diff --git a/romancal/regtest/test_wfi_16resultants.py b/romancal/regtest/test_wfi_16resultants.py
@@ -2,7 +2,6 @@
 
 import pytest
 import roman_datamodels as rdm
-from metrics_logger.decorators import metrics_logger
 
 from romancal.pipeline.exposure_pipeline import ExposurePipeline
 
@@ -16,7 +15,6 @@ def passfail(bool_expr):
 
 @pytest.mark.bigdata
 @pytest.mark.soctests
-@metrics_logger("DMS413")
 def test_16resultants_image_processing(rtdata, ignore_asdf_paths):
  """Tests for imaging processing requirements for 16 resultants (DMS413)"""
  # The input data is from INS for stress testing at some point this should be generated
@@ -72,7 +70,6 @@ def test_16resultants_image_processing(rtdata, ignore_asdf_paths):
 
 @pytest.mark.bigdata
 @pytest.mark.soctests
-@metrics_logger("DMS414")
 def test_16resultants_spectral_processing(rtdata, ignore_asdf_paths):
  """Tests for imaging processing requirements for 16 resultants (DMS413)"""
  # The input data is from INS for stress testing at some point this should be generated

diff --git a/romancal/regtest/test_wfi_dq_init.py b/romancal/regtest/test_wfi_dq_init.py
@@ -4,15 +4,13 @@
 
 import pytest
 import roman_datamodels as rdm
-from metrics_logger.decorators import metrics_logger
 
 from romancal.step import DQInitStep
 from romancal.stpipe import RomanStep
 
 from .regtestdata import compare_asdf
 
 
-@metrics_logger("DMS25")
 @pytest.mark.bigdata
 def test_dq_init_image_step(rtdata, ignore_asdf_paths):
  """DMS25 Test: Testing retrieval of best ref file for image data,
@@ -68,7 +66,6 @@ def test_dq_init_image_step(rtdata, ignore_asdf_paths):
  assert diff.identical, diff.report()
 
 
-@metrics_logger("DMS25")
 @pytest.mark.bigdata
 def test_dq_init_grism_step(rtdata, ignore_asdf_paths):
  """DMS25 Test: Testing retrieval of best ref file for grism data,

diff --git a/romancal/regtest/test_wfi_flat_field.py b/romancal/regtest/test_wfi_flat_field.py
@@ -3,7 +3,6 @@
 import pytest
 import roman_datamodels as rdm
 from crds.core.exceptions import CrdsLookupError
-from metrics_logger.decorators import metrics_logger
 
 from romancal.step import FlatFieldStep
 from romancal.stpipe import RomanStep
@@ -69,7 +68,6 @@ def test_flat_field_grism_step(rtdata, ignore_asdf_paths):
  assert diff.identical, diff.report()
 
 
-@metrics_logger("DMS79")
 @pytest.mark.bigdata
 @pytest.mark.soctests
 def test_flat_field_crds_match_image_step(rtdata, ignore_asdf_paths):

diff --git a/romancal/regtest/test_wfi_photom.py b/romancal/regtest/test_wfi_photom.py
@@ -5,15 +5,13 @@
 import pytest
 import roman_datamodels as rdm
 from astropy import units as u
-from metrics_logger.decorators import metrics_logger
 
 from romancal.step import PhotomStep
 from romancal.stpipe import RomanStep
 
 from .regtestdata import compare_asdf
 
 
-@metrics_logger("DMS140")
 @pytest.mark.bigdata
 def test_absolute_photometric_calibration(rtdata, ignore_asdf_paths):
  """DMS140 Test: Testing application of photometric correction using

diff --git a/romancal/regtest/test_wfi_pipeline.py b/romancal/regtest/test_wfi_pipeline.py
@@ -7,7 +7,6 @@
 import pytest
 import roman_datamodels as rdm
 from gwcs.wcstools import grid_from_bounding_box
-from metrics_logger.decorators import metrics_logger
 from numpy.testing import assert_allclose
 from roman_datamodels.dqflags import pixel
 
@@ -26,7 +25,6 @@ def passfail(bool_expr):
 
 @pytest.mark.bigdata
 @pytest.mark.soctests
-@metrics_logger("DMS86", "DMS87", "DMS88", "DMS361", "DMS280")
 def test_level2_image_processing_pipeline(rtdata, ignore_asdf_paths):
  """Tests for flat field imaging processing requirements DMS86 & DMS 87"""
  input_data = "r0000101001001001001_01101_0001_WFI01_uncal.asdf"
@@ -273,7 +271,6 @@ def test_level2_image_processing_pipeline(rtdata, ignore_asdf_paths):
 
 @pytest.mark.bigdata
 @pytest.mark.soctests
-@metrics_logger("DMS278", "DMS90", "DMS91", "DMS9", "DMS365")
 def test_level2_grism_processing_pipeline(rtdata, ignore_asdf_paths):
  """Tests for flat field grism processing requirements DMS90, DMS91 and DMS 278"""
  input_data = "r0000201001001001001_01101_0001_WFI01_uncal.asdf"