Skip to content

Clarify language around datatype casting and delete empty cell in Polars lazy execution tutorial.#114

Merged
Haleshot merged 2 commits intomarimo-team:mainfrom
debajyotid2:polars_lazyframes
Jun 30, 2025
Merged

Clarify language around datatype casting and delete empty cell in Polars lazy execution tutorial.#114
Haleshot merged 2 commits intomarimo-team:mainfrom
debajyotid2:polars_lazyframes

Conversation

@debajyotid2
Copy link
Contributor

@debajyotid2 debajyotid2 commented Jun 30, 2025

📝 Summary

This PR addresses @etrotta's comments in the Polars tutorial for lazy execution (#40).

  1. it changes and add sentences to resolve the ambiguous meaning of the sentence "Unless specified, Polars defaults to the pl.String datatype for all data.",
  2. deletes an empty markdown cell.

📋 Checklist

  • I have included package dependencies in the notebook file using --sandbox
  • If adding a course, include a README.md
  • Keep language direct and simple.

Copy link
Contributor

@Haleshot Haleshot left a comment

Choose a reason for hiding this comment

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

Thanks for the correction 🚀

@Haleshot Haleshot merged commit c419769 into marimo-team:main Jun 30, 2025
1 check passed
@debajyotid2 debajyotid2 deleted the polars_lazyframes branch June 30, 2025 16:05
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.

2 participants