Skip to content

Fixes for corrupted NAM data feed and NAM directory access#1742

Open
jasonfleming wants to merge 3 commits into
masterfrom
issue-1731
Open

Fixes for corrupted NAM data feed and NAM directory access#1742
jasonfleming wants to merge 3 commits into
masterfrom
issue-1731

Conversation

@jasonfleming

Copy link
Copy Markdown
Collaborator

Resolves #1731 and #1738

@jasonfleming
jasonfleming marked this pull request as ready for review July 20, 2026 17:06
@jasonfleming
jasonfleming requested a review from wwlwpd as a code owner July 20, 2026 17:06
@jasonfleming

Copy link
Copy Markdown
Collaborator Author

I have tested this successfully for 30 hours with successful downloads. I removed the trailing slash fix before rebasing to the latest master.

Comment thread bin/get_nam_status.pl

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jasonfleming I think this is inadvertently reverting the change that was already made. This should be fixedable via rebasing with the lastest master or if you can confirm this was not intended, I can do it.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @wwlwpd sorry about that, I tried to avoid this by reverting the added slash and then rebased with latest master but it looks like it didn't work. I'll make this change manually and update the PR.

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.

Add data quality checks for upstream meteorological data feeds

2 participants