fix(table): accept ChartDataType in mo.ui.table to resolve pyright error when passing chart.value

[VishakBaddur]

Problem

mo.ui.altair_chart.value is typed as ChartDataType, which includes IntoLazyFrame, list[dict[str, Any]], and dict[str, list[Any]]. But mo.ui.table.__init__'s data parameter only declared IntoDataFrame, not the other three members. This caused pyright to flag the common pattern mo.ui.table(chart.value) as a type error, even though it works correctly at runtime. This is the inline type error visible in VS Code reported in issue #8292.

Fix

Added the missing types (IntoLazyFrame, list[dict[str, Any]], dict[str, list[Any]]) to:

• mo.ui.table.__init__'s data parameter type annotation
• TableData in dataframe.py

The on_change callback type annotation was kept consistent with _convert_value's actual return type and not widened.

No runtime behavior changes - this is a type annotation-only fix.

Verification

• Confirmed pyright errors on mo.ui.table(chart.value) before the fix, passes after
• All existing table tests pass

Note

The infinite re-run described in the issue could not be reproduced on current main and appears environment-specific (VS Code + Ruff LSP). This PR addresses the reproducible type error only.

[vercel]

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

Project	Deployment	Actions	Updated (UTC)
marimo-docs	Ready	Preview, Comment	May 25, 2026 10:20pm

[Light2Dark]

There are some type errors

[Copilot]

Pull request overview

This PR updates type annotations so mo.ui.table(...) can accept mo.ui.altair_chart.value (typed as ChartDataType) without triggering pyright errors, aligning the table API with the data shapes produced by reactive Altair charts.

Changes:

• Expanded mo.ui.table.__init__’s data parameter type to include row/column-oriented chart payloads and IntoLazyFrame.
• Expanded mo.ui.table.__init__’s on_change callback annotation accordingly.
• Updated the shared TableData type alias in dataframe.py to include IntoLazyFrame.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File	Description
marimo/_plugins/ui/_impl/utils/dataframe.py	Extends TableData to include IntoLazyFrame for shared typing across table utilities.
marimo/_plugins/ui/_impl/table.py	Widens the table constructor and callback type annotations to accept chart-derived data shapes.

[VishakBaddur]

Done. list[dict[str, Any]] and dict[str, list[Any]] are now included in TableData

[VishakBaddur]

Acknowledged. Users passing a lazy frame directly should use table.lazy() for proper lazy behavior, this is existing guidance and unchanged by this PR. The type annotation accurately reflects what the runtime accepts.

[Light2Dark]

thanks!

[github-actions]

🚀 Development release published. You may be able to view the changes at https://marimo.app?v=0.23.9-dev25