-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
docs: show creation of DFSchema #8132
Conversation
Thank you very much @wjones127 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is very helpful. Thanks @wjones127
Co-authored-by: Andy Grove <andygrove73@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for this contribution @wjones127 -- I think this will help avoid a bunch of confusion
/// assert!(df_schema.has_column(&column)); | ||
/// ``` | ||
/// | ||
/// # Converting back to Arrow schema |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @wjones127 this is super helpful
Which issue does this PR close?
Closes #8131.
Rationale for this change
I got stuck figuring this out, and noticed there was very little documentation here.
I don't think I yet understand enough about DFSchema to fully document it. For now I'm mostly using unqualified schemas. Therefore, I'm not trying to complete #7724 with this PR.
What changes are included in this PR?
Adds some initial examples.
Are these changes tested?
Examples run with doc tests.
Are there any user-facing changes?