Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug]: CMR audit slc - not finding CLSC products with regex "OPERA_L2_CSLC-S1?_IW_*_{cslc_acquisition_dt_str}Z_v*_*" #692

Closed
toandn-jpl opened this issue Dec 8, 2023 · 0 comments
Assignees
Labels
bug Something isn't working needs triage Issue that requires triage pcm.r02 PCM Release 2

Comments

@toandn-jpl
Copy link

toandn-jpl commented Dec 8, 2023

Checked for duplicates

Yes - I've already checked

Describe the bug

rtc_native_id_pattern = f'OPERA_L2_CSLC-S1?_IW_*_{cslc_acquisition_dt_str}Z_v*_*

the above regex unable to detect CSLC-S1 product generations.

What did you expect?

@philipjyoon made the following changes and need to check them into develop branch. Thank you.

<         rtc_native_id_pattern = f'OPERA_L2_CSLC-S1_*_{cslc_acquisition_dt_str}Z_*_S1*'
---
> 
>         rtc_native_id_pattern = f'OPERA_L2_CSLC-S1?_IW_*_{cslc_acquisition_dt_str}Z_v*_*'
206,207c203
<             acquisition_time = cmr_product[33:48]
<             print(f"{acquisition_time=}")
---
>             acquisition_time = cmr_product[40:55]
216c212
<             acquisition_time = native_id_pattern[19:34]
---
>             acquisition_time = native_id_pattern[23:38]

Reproducible steps

1.
2.
3.
...

Environment

- Version of this software [e.g. vX.Y.Z]
- Operating System: [e.g. MacOSX with Docker Desktop vX.Y]
...
@toandn-jpl toandn-jpl added bug Something isn't working needs triage Issue that requires triage labels Dec 8, 2023
@hhlee445 hhlee445 added the pcm.r02 PCM Release 2 label Dec 8, 2023
philipjyoon added a commit that referenced this issue Dec 8, 2023
…it. It works much better now but there likely remains other bugs
hhlee445 added a commit that referenced this issue Dec 8, 2023
Issue #691 #692: Fixes a typo and CSLC granule pattern in cmr slc aud…
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working needs triage Issue that requires triage pcm.r02 PCM Release 2
Projects
None yet
Development

No branches or pull requests

3 participants