-
-
Notifications
You must be signed in to change notification settings - Fork 21
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
docs: update README to include Met Office and ECMWF NWPs #795
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: aryan lamba <[email protected]>
logs are at /aws/ecs/consumer/nwp-consumer-ecmwf-uk and the variables are stated here |
Hey @peterdudfield i complete all change that you tell. So, can you have a look. |
Hey @peterdudfield i complete all the change that you told me. So have a look |
@peterdudfield have a look .from last week this is pending so can you review it |
@devsjc you ok to look at this? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @lambaaryan011 !
@peterdudfield can you review this ? |
Pull Request
Description
This PR updates the infrastructure to support pulling Numerical Weather Predictions (NWPs) from both the Met Office and ECMWF. This enhancement ensures broader and more comprehensive weather data availability for nowcasting and forecasting applications.
Fixes #467
How Has This Been Tested?
⇾ Verified data integration from both Met Office and ECMWF sources.
⇾ Ensured compatibility with existing nowcasting infrastructure.
⇾ Ran Terraform validation to confirm no deployment issues.
If your changes affect data processing, have you plotted any changes? i.e. have you done a quick sanity check?
Checklist: