-
Notifications
You must be signed in to change notification settings - Fork 185
feat: Add Schema.from_{native,<backend>}
#2957
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
Merged
Merged
Changes from all commits
Commits
Show all changes
59 commits
Select commit
Hold shift + click to select a range
54d39ee
feat: Add `Schema.from_arrow`
dangotbanned 8b3773a
feat: Add `Schema.from_polars`
dangotbanned 30932b9
feat: Add `Schema.from_pandas`
dangotbanned c6d41d4
test: Add `test_schema_from_polars`
dangotbanned 0188904
test: add `test_schema_from_arrow`
dangotbanned 0c5b7d5
test: wip `test_schema_from_pandas`
dangotbanned 451ba84
Merge remote-tracking branch 'upstream/main' into schema-from
dangotbanned 99bc1d6
be quiet mypy
dangotbanned a9e28bf
Merge remote-tracking branch 'upstream/main' into schema-from
dangotbanned ac376ac
test: fill out pandas tests
dangotbanned f694136
test: Branch time unit for pandas nightly
dangotbanned 0b4c05d
feat: Add dtype/schema guards
dangotbanned eff9049
feat: Add `Schema.from_native`
dangotbanned 9cb6ae2
Merge branch 'main' into schema-from
dangotbanned 536566c
test: `from_native` polars coverage
dangotbanned 33bf1b6
Merge branch 'main' into schema-from
dangotbanned f546a23
test: `from_native` pyarrow coverage
dangotbanned a2d615f
test: error coverage
dangotbanned 6bd8085
feat(DRAFT): Re-do pandas-like
dangotbanned ed12a1c
test: Move some of pandas-like to conftest
dangotbanned 6760edf
cov
dangotbanned 89df6dd
migrate some tests, remove old constructor
dangotbanned c5869be
more pandas-like, fix missing `np.dtype`
dangotbanned 0a19c61
reorder methods, add to api ref
dangotbanned a0d0c32
ci: add `Schema` to `check_api_reference`
dangotbanned 228f42a
test: do a fancy py->pl->nw->pa->nw->pl->nw->pl->py
dangotbanned 6b1a03f
Merge branch 'main' into schema-from
dangotbanned 96813ea
refactor: organize imports
dangotbanned b204f45
docs: Add non-example docs
dangotbanned 9f07071
feat: Allow empty mapping
dangotbanned 12e6389
chore(typing): ignore mypy for now
dangotbanned cf613b7
docs: Add examples (excl cudf)
dangotbanned af6baf1
Merge branch 'main' into schema-from
dangotbanned 51f34ee
Merge branch 'main' into schema-from
dangotbanned d5bb7b9
Merge branch 'main' into schema-from
dangotbanned ec71058
Merge branch 'main' into schema-from
dangotbanned ef107aa
revert: remove `from_{cudf,modin,pandas}`
dangotbanned 87894ef
docs: remove returns sections π₯³
dangotbanned 24c740c
Merge remote-tracking branch 'upstream/main' into schema-from
dangotbanned 99e4ffd
Merge branch 'main' into schema-from
dangotbanned e227bda
Merge branch 'main' into schema-from
dangotbanned 4966868
Merge branch 'main' into schema-from
dangotbanned c3ee75b
Merge branch 'main' into schema-from
dangotbanned 41353bc
Merge branch 'main' into schema-from
dangotbanned 9d89527
Merge remote-tracking branch 'upstream/main' into schema-from
dangotbanned 5059f5d
Merge branch 'main' into schema-from
dangotbanned ff8b24c
Merge branch 'main' into schema-from
dangotbanned 20b5c2d
Merge branch 'main' into schema-from
dangotbanned fc23471
Merge remote-tracking branch 'upstream/main' into schema-from
dangotbanned f61c750
Merge branch 'main' into schema-from
dangotbanned 06a4ef9
refactor: Pass `()` to `infer` to be more explicit
dangotbanned 8559d43
Merge remote-tracking branch 'upstream/main' into schema-from
dangotbanned fe8372f
Merge remote-tracking branch 'upstream/main' into schema-from
dangotbanned 33f67dd
revert: changes to `_pandas_like.utils`
dangotbanned 8ab552e
Merge branch 'main' into schema-from
dangotbanned 50b074b
Merge branch 'main' into schema-from
dangotbanned a5ac573
Merge remote-tracking branch 'upstream/main' into schema-from
dangotbanned a6264d1
refactor: Update for (#3038)
dangotbanned 5377373
refactor: why do 2 lines when 1 does the trick?
dangotbanned File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
I've made this parameter positional-only for flexibility on our end.
I don't feel strongly about keeping it that way, but we should probably rename it to
native_schemafor consistency elsewhere.E.g.
nw.from_nativevsnw.Schema.__init__distinguish native vs narwhals-levels