FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Add `flatten`, `flatten_with_order` and `into_flat` to arrays by barakugav · Pull Request #1397 · rust-ndarray/ndarray · GitHub

Add flatten, flatten_with_order and into_flat to arrays - #1397

Merged
bluss merged 1 commit into
rust-ndarray:masterfrom
barakugav:flatten
Aug 2, 2024
Merged

Add flatten, flatten_with_order and into_flat to arrays#1397
bluss merged 1 commit into
rust-ndarray:masterfrom
barakugav:flatten

Conversation

Copy link
Copy Markdown
Contributor

No description provided.

Comment thread src/impl_methods.rs Outdated
barakugav force-pushed the flatten branch 2 times, most recently from 342f1f3 to 692fae5 Compare July 11, 2024 09:41
Comment thread src/impl_methods.rs
Comment thread src/impl_methods.rs Outdated

bluss commented Jul 21, 2024

Copy link
Copy Markdown
Member

Maintainer pushed to fix to always use into_shape_clone (it only clones when necessary)

bluss commented Jul 21, 2024

Copy link
Copy Markdown
Member

clippy lints we'll fix in another PR

bluss commented Jul 28, 2024

Copy link
Copy Markdown
Member

Can now be rebased on master because clippy errors are fixed there.

bluss added this to the 0.16.0 milestone Aug 1, 2024
bluss changed the title Adds 'flatten()' method to array Add flatten, flatten_with_order and into_flat to arrays Aug 2, 2024
bluss enabled auto-merge August 2, 2024 19:23
bluss added this pull request to the merge queue Aug 2, 2024
Merged via the queue into rust-ndarray:master with commit 91ecb92 Aug 2, 2024
barakugav deleted the flatten branch August 6, 2024 12:28
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL