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.

Sign up for GitHub

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

Jump to bottom

#16225: Int32 support for abs #16226

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

#16225: Int32 support for abs #16226

wants to merge 3 commits into from

Conversation

umadevimcw
Copy link
Contributor

Ticket

#16225

Problem description

Abs forge test failing for int32

What's changed

supported int32 support for abs op

Checklist

  • Post commit CI passes
  • Blackhole Post commit (if applicable)
  • Model regression CI testing passes (if applicable)
  • Device performance regression CI testing passes (if applicable)
  • (For models and ops writers) Full new models tests passes
  • New/Existing tests provide coverage for changes

@umadevimcw umadevimcw force-pushed the umadevimcw/forge_abs_test branch 2 times, most recently from 8f4b091 to 6624dfc Compare December 20, 2024 07:18
@umadevimcw umadevimcw force-pushed the umadevimcw/forge_abs_test branch from 6624dfc to 88b0c33 Compare December 20, 2024 09:25
@umadevimcw umadevimcw changed the title #16225: Support int32 support for abs in WHB0 #16225: Int32 support for abs Dec 20, 2024
@umadevimcw umadevimcw force-pushed the umadevimcw/forge_abs_test branch from 55462df to aaffcff Compare December 20, 2024 10:14
@umadevimcw umadevimcw force-pushed the umadevimcw/forge_abs_test branch from aaffcff to 89c8421 Compare December 20, 2024 12:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@patrickroberts patrickroberts Awaiting requested review from patrickroberts patrickroberts will be requested when the pull request is marked ready for review patrickroberts is a code owner

@yan-zaretskiy yan-zaretskiy Awaiting requested review from yan-zaretskiy yan-zaretskiy will be requested when the pull request is marked ready for review yan-zaretskiy is a code owner

@eyonland eyonland Awaiting requested review from eyonland eyonland will be requested when the pull request is marked ready for review eyonland is a code owner

@davorchap davorchap Awaiting requested review from davorchap davorchap will be requested when the pull request is marked ready for review davorchap is a code owner

@mywoodstock mywoodstock Awaiting requested review from mywoodstock mywoodstock will be requested when the pull request is marked ready for review mywoodstock is a code owner

@rtawfik01 rtawfik01 Awaiting requested review from rtawfik01 rtawfik01 will be requested when the pull request is marked ready for review rtawfik01 is a code owner

@rdjogoTT rdjogoTT Awaiting requested review from rdjogoTT rdjogoTT will be requested when the pull request is marked ready for review rdjogoTT is a code owner

@ttmtrajkovic ttmtrajkovic Awaiting requested review from ttmtrajkovic ttmtrajkovic will be requested when the pull request is marked ready for review ttmtrajkovic is a code owner

At least 0 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@umadevimcw