Skip to content

update system prompt & claude.md #6437

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 4 commits into from
Sep 19, 2025
Merged

update system prompt & claude.md #6437

merged 4 commits into from
Sep 19, 2025

Conversation

@Light2Dark
Copy link
Contributor

@Light2Dark Light2Dark commented Sep 19, 2025 •
edited
Loading

📝 Summary

  • Use polars instead of pandas
  • Altair instead of pyplot
  • Add a short sql cell guide (don't return df, pass in engine name)

🔍 Description of Changes

📋 Checklist

  • I have read the contributor guidelines.
  • 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).
  • I have added tests for the changes made.
  • I have run the code and verified that it works as expected.

@vercel
Copy link

vercel bot commented Sep 19, 2025 •
edited
Loading

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

Project Deployment Preview Comments Updated (UTC)
marimo-docs Ready Ready Preview Comment Sep 19, 2025 0:30am

@github-actions github-actions bot added the documentation Improvements or additions to documentation label Sep 19, 2025
@Light2Dark Light2Dark changed the title system prompt: use polars instead of pd, altair instead of pyplot, add sql cell guide system prompt & claude md: use polars instead of pd, altair instead of pyplot, add sql cell guide Sep 19, 2025
@Light2Dark Light2Dark changed the title system prompt & claude md: use polars instead of pd, altair instead of pyplot, add sql cell guide system prompt & claude.md: use polars instead of pd, altair instead of pyplot, add sql cell guide Sep 19, 2025
@Light2Dark Light2Dark changed the title system prompt & claude.md: use polars instead of pd, altair instead of pyplot, add sql cell guide update system prompt & claude.md Sep 19, 2025
@Light2Dark Light2Dark marked this pull request as ready for review September 19, 2025 10:20
mscolnick
mscolnick reviewed Sep 19, 2025
View reviewed changes
docs/_static/CLAUDE.md Outdated
import seaborn as sns
import polars as pl
import altair as alt
from vega_datasets import data
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe we can remove vega_datasets and instead do read_cab from either hf:// or GitHub

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I changed one example to use huggingface. We have this under best practices, so I think we can keep the other example to use vega_datasets.

- Consider using `vega_datasets` for common example datasets

@Light2Dark Light2Dark merged commit 3e80dc9 into main Sep 19, 2025
26 of 41 checks passed
@Light2Dark Light2Dark deleted the sham/use-polars-altair-system-prompt branch September 19, 2025 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mscolnick mscolnick mscolnick approved these changes

@akshayka akshayka Awaiting requested review from akshayka akshayka is a code owner

@manzt manzt Awaiting requested review from manzt manzt is a code owner

Assignees

No one assigned

Labels

documentation Improvements or additions to documentation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Light2Dark @mscolnick