Skip to content

Add batch_constant comparison and shift#1208

Merged
serge-sans-paille merged 1 commit intoxtensor-stack:masterfrom
AntoinePrv:constant-comp
Nov 18, 2025
Merged

Add batch_constant comparison and shift#1208
serge-sans-paille merged 1 commit intoxtensor-stack:masterfrom
AntoinePrv:constant-comp

Conversation

@AntoinePrv
Copy link
Copy Markdown
Contributor

@AntoinePrv AntoinePrv commented Nov 17, 2025

  • Add batch_constant comparison operators**
  • Fix size_t not being C
  • Add shift operators

constexpr bool operator()(T x, T y) const { return x != y; }
};
struct boolean_gt
{
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Contributor Author

@AntoinePrv AntoinePrv Nov 17, 2025

Choose a reason for hiding this comment

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

I wanted to but they are only constexpr since C++14

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Indeed!

@AntoinePrv AntoinePrv changed the title Add batch_constant comparison Add batch_constant comparison and shift Nov 17, 2025
@AntoinePrv
Copy link
Copy Markdown
Contributor Author

@serge-sans-paille this is ready for me if it is for you

@DiamonDinoia
Copy link
Copy Markdown
Contributor

This is complementary to #1162 that adds other binary ops. Might cause some merge conflicts.

@serge-sans-paille serge-sans-paille merged commit cadcd8f into xtensor-stack:master Nov 18, 2025
60 checks passed
@AntoinePrv AntoinePrv deleted the constant-comp branch November 18, 2025 09:07
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.

3 participants