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.

Sign up for GitHub

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

Jump to bottom

Present day missing from ERDDAP depth-averaged currents dataset #112

Closed
douglatornell opened this issue Oct 4, 2022 · 1 comment · Fixed by #113
Closed

Present day missing from ERDDAP depth-averaged currents dataset #112

douglatornell opened this issue Oct 4, 2022 · 1 comment · Fixed by #113
Assignees
@douglatornell
Labels
bug Something isn't working critical Model Results Workers
Milestone
v22.1

Comments

@douglatornell
Copy link
Member

douglatornell commented Oct 4, 2022 •
edited
Loading

Reported 30-Sep-2022 by Derek White (Project Manager with the Vancouver Fraser Port Authority ECHO program).

Note that 2022-09-30 dates/times and values are missing in this screenshot of an ERDDAP data request result from prior to 09:11 (time of Derek's email report) on the morning of 30-Sep-2022:
image
@douglatornell douglatornell added this to the v22.1 milestone Oct 4, 2022
@douglatornell douglatornell self-assigned this Oct 4, 2022
@douglatornell
Copy link
Member Author

I did the same query as Derek at about 11:30 on 30-Sep-2022 and found that the dates/times and values were present in the dataset.

I suspect that the root cause of the problem is a bug in the update_forecast_datasets worker or in the arg values it is being launched with after the NEMO forecast2 run.

I did a similar query at about 07:45 on 1-Oct-2022 and found that the dates/times and values for that day were missing. They appeared after update_forecast_datasets ran for that day's NEMO forecast run. That lends weight to the suspicion above.

@douglatornell douglatornell mentioned this issue Oct 4, 2022
Merged
@douglatornell douglatornell linked a pull request Oct 4, 2022 that will close this issue
Fix bug in update_forecast_datasets worker #113
Merged
douglatornell added a commit that referenced this issue Oct 5, 2022
@douglatornell
Fix bug re: rolling fcst missing today's CHS_currents.nc
d795c35 
re: issue #112
douglatornell added a commit that referenced this issue Oct 5, 2022
@douglatornell
Improve exclusion of files from rolling fcst tree
bf12473 
VENUS_*.nc files are excluded because they are not published on ERDDAP.
SalishSea_1d*.nc and *restart.nc files are now excluded from all directories
in the rolling forecast tree.

Closes #112
douglatornell added a commit that referenced this issue Oct 5, 2022
@douglatornell
Fix bug re: rolling fcst missing today's CHS_currents.nc
a97e18d 
re: issue #112
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@douglatornell douglatornell

Labels
bug Something isn't working critical Model Results Workers
Projects
None yet
Milestone
v22.1
Development

Successfully merging a pull request may close this issue.

Fix bug in update_forecast_datasets worker SalishSeaCast/SalishSeaNowcast
1 participant
@douglatornell