Skip to content
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.

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

feat: ✨ add mapping from Frictionless data types to Polars data types #1106

Merged
merged 1 commit into from
Mar 6, 2025

Conversation

martonvago
Copy link
Contributor

Description

This PR adds the mapping from Frictionless data types to Polars data types described in #1098.

I'm adding this now because I'm using it in the checks and presumably Signe will use it when writing to Parquet.

This PR needs a quick review.

Checklist

  • Added or updated tests
  • Updated documentation
  • Ran just run-all

@martonvago martonvago self-assigned this Mar 5, 2025
@martonvago martonvago marked this pull request as ready for review March 5, 2025 10:10
@martonvago martonvago requested a review from a team as a code owner March 5, 2025 10:10
Copy link
Member

@lwjohnst86 lwjohnst86 left a comment

Choose a reason for hiding this comment

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

Easy peasy!!

Copy link
Member

@signekb signekb left a comment

Choose a reason for hiding this comment

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

Very nice 👌

@signekb signekb merged commit eca3dbd into main Mar 6, 2025
4 checks passed
@signekb signekb deleted the feat/frictionless-polars-mapping branch March 6, 2025 12:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants