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

[FRONTEND] Fix assertion containing incorrect function call in min/max #3358

Merged
merged 2 commits into from
Mar 13, 2024

Conversation

prathams417
Copy link
Contributor

@prathams417 prathams417 commented Mar 12, 2024

When running with assertions (TRITON_DEBUG=1), there was a compilation issue in the min and max functions in standard.py. Previously, to check whether the datatype was an integer, a function call to input.dtype.is_integer_type() was made, but this function call does not exist as a part of dtype. This has now been replaced with the appropriate function call input.dtype.is_int().

@prathams417 prathams417 requested a review from ptillet as a code owner March 12, 2024 19:46
@prathams417 prathams417 force-pushed the prathams/fix-min-max-assertion branch from 1f8104b to 3f239b3 Compare March 12, 2024 19:53
@Jokeren Jokeren changed the title Fix assertion containing incorrect function call in min/max [FRONTEND] Fix assertion containing incorrect function call in min/max Mar 12, 2024
@Jokeren Jokeren merged commit 4e8e389 into triton-lang:main Mar 13, 2024
4 checks passed
@prathams417 prathams417 deleted the prathams/fix-min-max-assertion branch March 13, 2024 14:51
htyu pushed a commit to htyu/triton that referenced this pull request Mar 20, 2024
triton-lang#3358)

When running with assertions (`TRITON_DEBUG=1`), there was a compilation
issue in the min and max functions in standard.py. Previously, to check
whether the datatype was an integer, a function call to
`input.dtype.is_integer_type()` was made, but this function call does
not exist as a part of dtype. This has now been replaced with the
appropriate function call `input.dtype.is_int()`.

Co-authored-by: Keren Zhou <kerenzhou@openai.com>
karupayun pushed a commit to openxla/triton that referenced this pull request Apr 3, 2024
triton-lang#3358)

When running with assertions (`TRITON_DEBUG=1`), there was a compilation
issue in the min and max functions in standard.py. Previously, to check
whether the datatype was an integer, a function call to
`input.dtype.is_integer_type()` was made, but this function call does
not exist as a part of dtype. This has now been replaced with the
appropriate function call `input.dtype.is_int()`.

Co-authored-by: Keren Zhou <kerenzhou@openai.com>
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