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

fix #1219 convert v2 time offsets to sr3 correctly #1222

Closed
wants to merge 1 commit into from

Conversation

petersilva
Copy link
Contributor

not the most general way to fix it, but via find and grep... I only found -1D, and -2D... so that's really all the cases we need to take care of... anyways... does enough of the job.

Copy link

Test Results

248 tests   247 ✅  1m 27s ⏱️
  1 suites    1 💤
  1 files      0 ❌

Results for commit 6673508.

@petersilva petersilva marked this pull request as ready for review September 16, 2024 20:05
@petersilva
Copy link
Contributor Author

the flow test failures we see have nothing to do with the branch.

@petersilva
Copy link
Contributor Author

@reidsunderland surmised the patch does not work if the strings do not end with the date pattern... did some tests... he is right, this does not fix that case.


directory /tmp/hpfx_amis/${HOHO}/${YYYYMMDD-1D}/lala

different approach needed.

@petersilva
Copy link
Contributor Author

broken existing configs:
'''

./ddsr_science20/poll/hpfx-dfo.conf:directory /data/dd/public_data/dfo/pops_model/${YYYYMMDD-1D}/canso100
./ddsr-stage/poll/class_noaa.conf:directory /${YYYYMMDD-1D}/VIIRS-SDR/VIIRS-Moderate-Resolution-Band-01-SDR/J01/

'''
on the other hand... all occurrences of date offsets are either ${YYYYMMDD-1D} or ${YYYYMMDD-2D} ... so could just replace those
two strings outright and it would work for all extant cases.

@petersilva
Copy link
Contributor Author

The original syntax was meant to work in other contexts... but it looks like only the full date is actually used anywhere.

@petersilva
Copy link
Contributor Author

went a different way... other way already merged.

@petersilva petersilva closed this Sep 18, 2024
@petersilva petersilva deleted the issue1219 branch September 18, 2024 20:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant