Fix date summary polars #1146

skrawcz · 2024-09-19T05:01:45Z

Fixes a few polars things

Changes

PR has an informative and human-readable title (this will be pulled into the release notes)
Changes are limited to a single goal (no scope creep)
Code passed the pre-commit check & code is left cleaner/nicer than when first encountered.
Any change in functionality is tested
New functions are documented (with a description, list of inputs, and expected output)
Placeholder code is flagged / future TODOs are captured in comments
Project documentation has been updated if adding/changing functionality.

Didn't have appropriate test coverage, now we do.

If they're empty it breaks the UI.

So it's more up to date to use.

To push out polars fix.

elijahbenizzy

Looks good, we're hardcoding std to 0 but that's OK Fixing tests

Not needed -- doesn't make sense for dates with polars.

skrawcz and others added 5 commits September 18, 2024 22:00

Fixes handling datetime polars bug

6effbe0

Didn't have appropriate test coverage, now we do.

Fixes empty quantiles

aa9592a

If they're empty it breaks the UI.

Updates Polars notebook to use magic

a61c232

So it's more up to date to use.

Bumps SDK to 0.7.2

75a019e

To push out polars fix.

Bumps hamilton server/frontend version from 0.0.14 to 0.0.15

219aaac

elijahbenizzy approved these changes Sep 19, 2024

View reviewed changes

Removing test

ce9d5db

Not needed -- doesn't make sense for dates with polars.

elijahbenizzy merged commit fa049fb into main Sep 19, 2024
24 of 30 checks passed

elijahbenizzy deleted the fix_date_summary_polars branch September 19, 2024 05:36