-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
[namedarray] split .set_dims()
into .expand_dims()
and broadcast_to()
#8380
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
71 commits
Select commit
Hold shift + click to select a range
1224b54
add `.set_dims()`, `.transpose()` and `.T` to namedarray
andersy005 409c5de
more typying fixes
andersy005 43e10d8
more typing fixes
andersy005 d6c7758
override set_dims for IndexVariable
andersy005 a920e11
fix dims
andersy005 e765d7d
split `.set_dims()` into `.expand_dims()` and `broadcast_to()`
andersy005 00504f4
more typing fixes
andersy005 5a06dec
update whats-new
andersy005 ec17489
update tests
andersy005 a245021
doc fixes
andersy005 447f226
update whats-new
andersy005 9ad56a9
keep `.set_dims()` on `Variable()`
andersy005 a5918c3
update docs
andersy005 8091265
revert to set_dims
andersy005 2b93273
revert to .set_dims on Variable
andersy005 7ea1fb3
Update xarray/namedarray/core.py
andersy005 456d57c
restore .transpose on variable
andersy005 3f458c8
revert to set_dims in Variable
andersy005 c641ee6
Merge branch 'main' into add-set-dims
andersy005 aae2861
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] d6240de
fix docstring
andersy005 1364345
update test_namedarray
andersy005 b793f74
update tests
andersy005 b15971f
Merge branch 'main' into add-set-dims
andersy005 cc67b63
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 79119ce
Merge branch 'main' into add-set-dims
andersy005 f156744
Merge branch 'main' into add-set-dims
andersy005 ab6262b
Apply suggestions from code review
andersy005 6fbf90d
Merge branch 'main' into add-set-dims
andersy005 069c353
fix formatting issue
andersy005 5e1af7a
fix tests
andersy005 2499c93
update expand_dims
andersy005 3e4d8fa
update tests
andersy005 64b674a
Merge branch 'main' into add-set-dims
andersy005 ff01bce
update tests
andersy005 2c4b2b6
remove unnecessary guard conditions
andersy005 c0aefaa
Update type hints in NamedArray class and test cases
andersy005 e2cce05
Refactor NamedArray T property to handle non-2D arrays
andersy005 3633a2e
Reverse the order of dimensions in x.T
andersy005 326dad4
Refactor broadcasting and dimension expansion in NamedArray
andersy005 1c562a9
update docstring
andersy005 a7fd5c7
add todo item
andersy005 ff1a6fe
Merge branch 'main' into add-set-dims
andersy005 82e89c0
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 47121ef
Merge branch 'main' into add-set-dims
andersy005 31acb3b
use comprehension
andersy005 6fdbad6
use dim
andersy005 f1feb9f
Merge branch 'main' into add-set-dims
andersy005 401b6d5
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] c769639
fix imports
andersy005 dd64dbf
Merge branch 'main' into add-set-dims
andersy005 06cba49
Merge branch 'main' into add-set-dims
andersy005 959d97c
formatting only
andersy005 7be6a2d
Apply suggestions from code review
andersy005 3dfbce4
Merge branch 'main' into add-set-dims
andersy005 ea24613
[skip-rtd] fix indentation
andersy005 3fba906
Merge branch 'main' into add-set-dims
andersy005 f2a5989
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 8f494d3
Merge branch 'main' into add-set-dims
andersy005 5f7e5c5
refactor expand_dims to simplify API
andersy005 ce35128
Merge branch 'main' into add-set-dims
andersy005 0fb4443
fix type hint for `dim` parameter in test.
andersy005 a831c14
fix typing issues
andersy005 d880621
Merge branch 'main' into add-set-dims
andersy005 1524519
fix UnboundLocalError: local variable 'flattened_dims' referenced bef…
andersy005 e6af928
fix type hint
andersy005 7c3de30
Merge branch 'main' into add-set-dims
andersy005 a6ad9b8
ignore typing
andersy005 fd1c0d3
update whats-new
andersy005 6ec1170
Merge branch 'main' into add-set-dims
andersy005 e84abf9
adjust the `broadcast_to` method to prohibit adding new dimensions, a…
andersy005 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
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.
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.
Uh oh!
There was an error while loading. Please reload this page.