Skip to content

fix: ignore warnings for duration in column summaries. - #10531

Open
kjgoodrick wants to merge 1 commit into
marimo-team:mainfrom
kjgoodrick:fix_duration
Open

fix: ignore warnings for duration in column summaries.#10531
kjgoodrick wants to merge 1 commit into
marimo-team:mainfrom
kjgoodrick:fix_duration

Conversation

@kjgoodrick

Copy link
Copy Markdown
Contributor

Binning duration columns is expected to fail and the fallback still produces the chart as expected. This change stops the warning from being emitted if the column type is a duration. I think a more robust fix would require support for durations/timedeltas in Vega-lite. For now the chart is good enough that emitting the warning is not helpful.

📝 Summary

Closes #10530

📋 Pre-Review Checklist

  • For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on Discord, or the community discussions (Please provide a link if applicable).
  • Any AI generated code has been reviewed line-by-line by the human PR author, who stands by it.
  • Video or media evidence is provided for any visual changes (optional).

✅ Merge Checklist

  • I have read the contributor guidelines.
  • Documentation has been updated where applicable, including docstrings for API changes.
  • Tests have been added for the changes made.

These are expected to fail and the fallback still produces the chart as expected
@vercel

vercel Bot commented Aug 13, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
marimo-docs Ready Ready Preview Aug 13, 2026 4:55am

Request Review

@github-actions

Copy link
Copy Markdown
Contributor

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

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.

Duration column shows warning in dataframe viewer

1 participant