Skip to content

fix and test sorting arrays with >1 dimension and custom similar#49392

Merged
LilithHafner merged 3 commits intomasterfrom
sort-custom-matrix-bugfix
May 31, 2023
Merged

fix and test sorting arrays with >1 dimension and custom similar#49392
LilithHafner merged 3 commits intomasterfrom
sort-custom-matrix-bugfix

Conversation

@LilithHafner
Copy link
Copy Markdown
Member

This bug only comes up when sorting AbstractArrays that are neither AbstractVectors nor Arrays and define a similar(A, dim::Int) that does not return a Vector. It was introduced in 1.9, almost certainly my fault.

@LilithHafner LilithHafner added bugfix This change fixes an existing bug sorting Put things in order backport 1.9 Change should be backported to release-1.9 labels Apr 17, 2023
@LilithHafner
Copy link
Copy Markdown
Member Author

I'm going to merge this bugfix in a couple days unless someone objects.

@LilithHafner LilithHafner merged commit d5145de into master May 31, 2023
@LilithHafner LilithHafner deleted the sort-custom-matrix-bugfix branch May 31, 2023 13:44
@KristofferC KristofferC mentioned this pull request Jun 6, 2023
36 tasks
@KristofferC KristofferC removed the backport 1.9 Change should be backported to release-1.9 label Jul 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix This change fixes an existing bug sorting Put things in order

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants