Skip to content

Conversation

@parsabee
Copy link
Contributor

As mentioned in this issue: #50521 (comment), tf-mlir-translate didn't have a mechanism for handling data type with subtypes such as DT_RESOURCE. This patch adds support for handling types that might have subtypes to tf-mlir-translate and it's python wrapper.

@google-ml-butler google-ml-butler bot added the size:L CL Change Size: Large label Apr 28, 2022
@gbaned gbaned requested a review from joker-eph April 29, 2022 04:00
@google-ml-butler google-ml-butler bot added the awaiting review Pull request awaiting review label Apr 29, 2022
@parsabee parsabee marked this pull request as draft April 29, 2022 15:07
@parsabee parsabee marked this pull request as ready for review April 29, 2022 15:07
@parsabee parsabee marked this pull request as draft April 29, 2022 15:07
@parsabee parsabee marked this pull request as ready for review April 29, 2022 15:15
@joker-eph joker-eph requested a review from ChiaHungDuan May 2, 2022 06:00
@Mogball Mogball requested review from Mogball May 2, 2022 17:14
Copy link

@Mogball Mogball left a comment

Choose a reason for hiding this comment

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

After these changes, LGTM

Copy link

@Mogball Mogball left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for the patch!

@google-ml-butler google-ml-butler bot added kokoro:force-run Tests on submitted change ready to pull PR ready for merge process labels May 3, 2022
@kokoro-team kokoro-team removed the kokoro:force-run Tests on submitted change label May 3, 2022
@google-ml-butler google-ml-butler bot removed the ready to pull PR ready for merge process label May 3, 2022
@tensorflowbutler tensorflowbutler removed the awaiting review Pull request awaiting review label May 5, 2022
@gbaned gbaned requested review from Mogball and removed request for Mogball May 5, 2022 14:51
@google-ml-butler google-ml-butler bot added the awaiting review Pull request awaiting review label May 5, 2022
@google-ml-butler google-ml-butler bot added kokoro:force-run Tests on submitted change ready to pull PR ready for merge process labels May 5, 2022
@kokoro-team kokoro-team removed the kokoro:force-run Tests on submitted change label May 5, 2022
@tensorflowbutler tensorflowbutler removed the awaiting review Pull request awaiting review label May 7, 2022
@gbaned gbaned added ready to pull PR ready for merge process and removed ready to pull PR ready for merge process labels May 17, 2022
@copybara-service copybara-service bot merged commit ce0662f into tensorflow:master May 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready to pull PR ready for merge process size:L CL Change Size: Large

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants