Opened 9 months ago

Closed 8 months ago

#2803 closed defect (fixed)

resumer for sentinel1 recipe has wrong coverage id

Reported by: Dimitar Misev Owned by: Bang Pham Huu
Priority: major Milestone: 10.3
Component: wcst_import Version: 10.3
Keywords: Cc:
Complexity: Medium

Description

A sentinel1 recipe has coverage_id with some variables, such as:

    "coverage_id": "S1_GRDH_${modebeam}_DV_${polarisation}",
    "modebeams": ["IW", "EW", "S1", "S2", "S3", "S4", "S5", "S6"],
    "polarisations": ["VH"],

This imports data into multiple coverages

  • S1_GRDH_IW_DV_VH
  • S1_GRDH_EW_DV_VH

The problem is that only one resumer is created per ingredient and it has name S1_GRDH_\$\{modebeam\}_DV_\$\{polarisation\}.resume.json. Probably there should be one resumer for the actual imported coverage_ids (S1_GRDH_IW_DV_VH, S1_GRDH_EW_DV_VH, etc) instead of the template coverage_id.

When rerunning wcst_import on the same json, it gives error:

        if coverage_id not in Resumer.__IMPORTED_DATA_DICT:
            resume_file_path = Resumer.get_resume_file_path(ConfigManager.resumer_dir_path, coverage_id)
            Resumer.__RESUMER_FILE_NAME_DICT[coverage_id] = resume_file_path
            try:
                if os.path.isfile(resume_file_path) \
                        and os.access(resume_file_path, os.R_OK):
                    from util.coverage_util import CoverageUtilCache
                    cov = CoverageUtilCache.get_cov_util(coverage_id)

To reproduce: there's a systemtest 023-sentinel1_GRD_custom_recipe, enable resumer in it and rerun it twice or so.

Change History (1)

comment:1 by Dimitar Misev, 8 months ago

Resolution: fixed
Status: assignedclosed
Note: See TracTickets for help on using tickets.