-
Notifications
You must be signed in to change notification settings - Fork 9
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
Partition weather station model predictions table, part 1 #4059
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #4059 +/- ##
=======================================
Coverage 80.37% 80.37%
=======================================
Files 309 309
Lines 11841 11841
Branches 537 537
=======================================
Hits 9517 9517
Misses 2138 2138
Partials 186 186 ☔ View full report in Codecov by Sentry. |
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.
Nice work, looks good. I imagine they'll be some down time and an increase in db size temporarily with all the indexes being generated?
if [ -z ${PG_PASSWORD+0} ] | ||
then | ||
echo "PG_PASSWORD not specified" | ||
echo "Specify a postgress password" |
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.
super nit: extra 's' on postgres in a few places here
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.
Done in: 9c0393e
- name: "TABLE" | ||
description: "The table that is partitioned" | ||
required: true | ||
value: "prod" |
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.
Is this intentionally 'prod'?
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.
Copy-pasta thanks for catching
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.
Done in: 9c0393e
Quality Gate passedIssues Measures |
weather_station_model_predictions
from the months we had data this past fire season, and 3 months into the futureTest Links:
Landing Page
MoreCast
Percentile Calculator
C-Haines
FireBat
FireBat bookmark
Auto Spatial Advisory (ASA)
HFI Calculator