Skip to content

Removing 2 digit duckdb rounding from fire_wind_risk_regional_agg#210

Closed
norlandrhagen wants to merge 5 commits intomainfrom
remove_rounding
Closed

Removing 2 digit duckdb rounding from fire_wind_risk_regional_agg#210
norlandrhagen wants to merge 5 commits intomainfrom
remove_rounding

Conversation

@norlandrhagen
Copy link
Collaborator

No description provided.

@norlandrhagen norlandrhagen added the e2e End-to-end testing label Sep 9, 2025
@norlandrhagen norlandrhagen marked this pull request as ready for review September 9, 2025 16:10
@andersy005 andersy005 added the QA/QC For reporting weird things spotted in the data label Sep 9, 2025
@andersy005
Copy link
Member

@norlandrhagen. does this change require any front-end coordination with @Shane98c / @katamartin ?

@katamartin
Copy link
Member

@norlandrhagen @andersy005 should we test this one on QA just to be safe?

@andersy005
Copy link
Member

@norlandrhagen @andersy005 should we test this one on QA just to be safe?

👍🏽 just deployed a coiled job

@katamartin
Copy link
Member

Thanks @andersy005!

Quick question so I understand, is this change is only live for qa? By default, nothing deploys for PRs to staging or production, right?

@andersy005
Copy link
Member

By default, nothing deploys for PRs to staging or production, right?

yep. we currently don't deploy PRs to any environment... if we want to automatically deploy PRs to qa, i can easily add a workflow to the repo

@katamartin
Copy link
Member

Thanks @andersy005 -- mainly just wanted to confirm to wrap my head around #212!

@andersy005
Copy link
Member

qa is live with the latest data from this branch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

e2e End-to-end testing QA/QC For reporting weird things spotted in the data

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants