Skip to content

Add UM Global-NL path to NWP data sources#408

Merged
katyalmohit merged 2 commits intomainfrom
update-nwp-list
Mar 27, 2026
Merged

Add UM Global-NL path to NWP data sources#408
katyalmohit merged 2 commits intomainfrom
update-nwp-list

Conversation

@katyalmohit
Copy link
Copy Markdown
Contributor

@katyalmohit katyalmohit commented Mar 26, 2026

Pull Request

Description

Add UM Global-NL to NWP data paths.

Fixes #407

How Has This Been Tested?

Tested locally with AWS

Checklist:

  • My code follows OCF's coding style guidelines
  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked my code and corrected any misspellings

@peterdudfield
Copy link
Copy Markdown
Contributor

Did you run the dashboard locally and check it worked?

@katyalmohit
Copy link
Copy Markdown
Contributor Author

Did you run the dashboard locally and check it worked?

Earlier, I didn't update the lat_lon_datasets list, so that was throwing the error. Now, it's working.

@katyalmohit katyalmohit merged commit edd8d4d into main Mar 27, 2026
5 checks passed
@katyalmohit katyalmohit deleted the update-nwp-list branch March 27, 2026 08: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.

Add um-global_nl to nwp list

2 participants