Skip to content

Documentation Bugfix: Polars cannot read json from URL#9397

Merged
akshayka merged 1 commit into
marimo-team:mainfrom
daniel-bogdoll:patch-2
Apr 27, 2026
Merged

Documentation Bugfix: Polars cannot read json from URL#9397
akshayka merged 1 commit into
marimo-team:mainfrom
daniel-bogdoll:patch-2

Conversation

@daniel-bogdoll

@daniel-bogdoll daniel-bogdoll commented Apr 27, 2026

Copy link
Copy Markdown
Contributor

Affected documentation: https://docs.marimo.io/guides/working_with_data/dataframes/#__tabbed_1_2

Polars read_json() cannot read from a URL. Documentation: https://docs.pola.rs/api/python/dev/reference/api/polars.read_json.html

📝 Summary

The current implementation throws the following error:

FileNotFoundError
No such file or directory (os error 2): https://raw.githubusercontent.com/vega/vega-datasets/master/data/cars.json

The fix does not introduce new dependencies.

Screenshot:
image

📋 Pre-Review Checklist

  • 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.

@vercel

vercel Bot commented Apr 27, 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, Comment Apr 27, 2026 2:27pm

Request Review

@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Apr 27, 2026
@github-actions

github-actions Bot commented Apr 27, 2026

Copy link
Copy Markdown

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

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 1 file

@daniel-bogdoll

Copy link
Copy Markdown
Contributor Author

I have read the CLA Document and I hereby sign the CLA

@daniel-bogdoll

Copy link
Copy Markdown
Contributor Author

recheck

@akshayka akshayka left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@akshayka akshayka merged commit e89912c into marimo-team:main Apr 27, 2026
28 of 30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants