Skip to content
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

hvncat: Ensure output ndims are >= the ndims of input arrays #41201

Merged
merged 20 commits into from
Jul 16, 2021

Conversation

BioTurboNick
Copy link
Contributor

Breaking down #41143 into smaller pieces.

This PR ensures the output ndims are at least as large as the ndims of the input arrays.

The max nd calculation caused allocations to blow up when I had it in the same method as the remaining code. Removing it to a separate method fixed that.

@BioTurboNick
Copy link
Contributor Author

This PR should probably have the 1.7 backport label too? @KristofferC @simeonschaub @JeffBezanson

@vtjnash
Copy link
Member

vtjnash commented Jul 15, 2021

needs rebase now

@simeonschaub simeonschaub added the bugfix This change fixes an existing bug label Jul 16, 2021
Copy link
Member

@simeonschaub simeonschaub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Sorry for the delay!

@simeonschaub simeonschaub merged commit a2f5fe5 into JuliaLang:master Jul 16, 2021
@BioTurboNick
Copy link
Contributor Author

No worries, the delay was necessary to make sure it would merge correctly

KristofferC pushed a commit that referenced this pull request Jul 19, 2021
KristofferC pushed a commit that referenced this pull request Jul 20, 2021
petvana added a commit to petvana/julia that referenced this pull request Aug 5, 2021
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants