Skip to content

fix: SECOND-based DATETIME_DIFF in urine_output_rate#2056

Open
Chessing234 wants to merge 3 commits into
MIT-LCP:mainfrom
Chessing234:fix/1549-urine-output-rate-datetime-diff
Open

fix: SECOND-based DATETIME_DIFF in urine_output_rate#2056
Chessing234 wants to merge 3 commits into
MIT-LCP:mainfrom
Chessing234:fix/1549-urine-output-rate-datetime-diff

Conversation

@Chessing234

Copy link
Copy Markdown
Contributor

Summary

  • BigQuery truncates MINUTE/HOUR diffs; Postgres returns fractions
  • Use SECOND/60 and SECOND/3600 like kdigo_uo

Fixes #1549

Test plan

  • CI transpile / equivalence

Made with Cursor

Chessing234 and others added 3 commits July 16, 2026 17:39
Matches kdigo_uo so BigQuery and PostgreSQL agree on fractional hours (MIT-LCP#1549).

Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
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.

DATETIME_DIFF implementation varies between postgres and bigquery

1 participant