Skip to content

Simplifying argmin and argmax where possible#2499

Merged
JohanMabille merged 2 commits intoxtensor-stack:masterfrom
tdegeus:argmin
Mar 28, 2022
Merged

Simplifying argmin and argmax where possible#2499
JohanMabille merged 2 commits intoxtensor-stack:masterfrom
tdegeus:argmin

Conversation

@tdegeus
Copy link
Copy Markdown
Member

@tdegeus tdegeus commented Mar 25, 2022

Hopefully this avoids mysterious segmentation faults on Windows

Checklist

  • The title and commit message(s) are descriptive.
  • Small commits made to fix your PR have been squashed to avoid history pollution.
  • Tests have been added for new features or bug fixes.
  • API of new functions and classes are documented.

Description

@tdegeus tdegeus closed this Mar 25, 2022
@tdegeus tdegeus reopened this Mar 25, 2022
@tdegeus tdegeus closed this Mar 25, 2022
@tdegeus tdegeus reopened this Mar 25, 2022
@tdegeus tdegeus added the Bug label Mar 27, 2022
@JohanMabille JohanMabille merged commit 1ef6192 into xtensor-stack:master Mar 28, 2022
@JohanMabille
Copy link
Copy Markdown
Member

Thanks!

@tdegeus tdegeus deleted the argmin branch March 28, 2022 09:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants