-
Notifications
You must be signed in to change notification settings - Fork 70
Implementation for complex sorting of dndarrays and take
#2482
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 43 commits
Commits
Show all changes
76 commits
Select commit
Hold shift + click to select a range
fb0ba46
- Extracted the exchange logic of `vectorized_sort`
Berkant03 1f85501
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] a886580
Fixed missing import
8e9342e
Add failing test
brownbaerchen b5c8633
Fix test
brownbaerchen cadff06
Specify device in a few places
brownbaerchen 26580db
Fix test for real this time
brownbaerchen fc25e35
Complex sort implementation.
29815be
Merge remote-tracking branch 'origin/vectorized_sort_gpu' into featur…
d500a9e
add device specification
33dade2
add original_split parameter to have correct reorder behaviour in vec…
d16e3ad
Merge remote-tracking branch 'origin/main' into features/2372-complex…
1e59936
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] fdb6104
- Removed stable parameter
52aeea6
- Added test for `reorder`
92ef942
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 7fd4218
- changed complex array generation
c4a3e51
- better numpy version handling
cda25d6
Merge remote-tracking branch 'origin/main' into features/2372-complex…
65744ac
removed stable parameter
64f9012
add information regarding sort stability.
26ecb92
Update tests/core/test_sorting.py
Berkant03 448417b
Update heat/core/manipulations.py
Berkant03 796b4a9
Update tests/core/test_sorting.py
Berkant03 67aaaf9
Update heat/core/manipulations.py
Berkant03 c677dd2
Update heat/core/manipulations.py
Berkant03 351b308
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] f5981a0
use `np.prod` instead of `math.prod`
5744525
use self communicator for local sort
db510e8
- Fixed indexing for reorder.
6a036c0
rename parameter
868dcc8
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 94d2b8e
update documentation for perameter renaming
55793c4
- Added None Split case for complex sorting
1c4a250
Update heat/core/manipulations.py
Berkant03 49ba668
Update heat/core/manipulations.py
Berkant03 c037d5b
Update heat/core/manipulations.py
Berkant03 cc74934
use isclose for array comparison
2f3535c
Merge remote-tracking branch 'origin/main' into features/2372-complex…
f66e893
removed and renamed parameter
b50ce27
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] a85066b
changed split axis for the non distributed case
60d03c1
Change the split of the index array
4305e94
Allow local reordering when not on axis != split
6dc072d
Merge remote-tracking branch 'origin/main' into features/2372-complex…
71aa8c4
remove the resplit from `reorder`
6398226
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 0d7c66d
Fix nameerrors due to api change
605d147
Merge remote-tracking branch 'origin/features/2372-complex_sort' into…
c5177b8
Update heat/core/manipulations.py
Berkant03 3e2123e
Update heat/core/manipulations.py
Berkant03 e16d7d8
Update heat/core/manipulations.py
Berkant03 91fe22e
Update heat/core/manipulations.py
Berkant03 a4fcf10
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 4af0aad
Update heat/core/manipulations.py
Berkant03 219c3ba
Update tests/core/test_sorting.py
Berkant03 4104d9e
Update heat/core/manipulations.py
Berkant03 1ff8355
- changed pos of the clone
fbee41b
renaming or `reorder` to `take`
b64f446
Merge remote-tracking branch 'origin/main' into features/2372-complex…
2a80883
update documentation.
65e2942
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 9132d4e
removed assertion, to allow `np.take` like behaviour
9f9075f
Merge remote-tracking branch 'origin/features/2372-complex_sort' into…
7ec9262
allow None Axis and similar behaviour to `np.take`
c488015
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] b6286fe
add some device checks
9b6db04
add more split assertions
82f7c57
- Fixed distributed take for any length index tensors.
82bf2a9
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 5a9bff3
add more parameter checks
b880bbc
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] c8b4ec9
Merge branch 'main' into features/2372-complex_sort
brownbaerchen 6411df4
Change assert for `test_take`
b1d6caf
Merge remote-tracking branch 'origin/features/2372-complex_sort' into…
b6ca8ec
fix test case for 1 process
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
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.