Skip to content

[Bridgetower] Fix for Nonetype in image#39

Closed
MohitIntel wants to merge 3 commits into
habana-mainfrom
mdeopujari/SW-174850
Closed

[Bridgetower] Fix for Nonetype in image#39
MohitIntel wants to merge 3 commits into
habana-mainfrom
mdeopujari/SW-174850

Conversation

@MohitIntel
Copy link
Copy Markdown

Fixes [SW-174850] Nonetype error in image.

Before submitting

  • [N ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
  • [N ] Did you make sure to update the documentation with your changes?
  • [N ] Did you write any new necessary tests?

@MohitIntel MohitIntel requested a review from a user February 14, 2024 02:21
@MohitIntel MohitIntel changed the base branch from main to habana-main February 14, 2024 02:22
Copy link
Copy Markdown

@vivekgoe vivekgoe left a comment

Choose a reason for hiding this comment

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

Please remove JIRA Id from commit message. Reason for this request is that when migrating PRs from OH-fork to OH main we want to make minimal changes (specially to commit message) and also avoid exposing habana internal information.

Copy link
Copy Markdown

@murthy-l-habana murthy-l-habana left a comment

Choose a reason for hiding this comment

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

Again, as mentioned in JIRA, this API is not exposed to user and should not be used for custom ops addition. Please use user level API only.

@libinta libinta changed the title [SW-174850][Bridgetower] Fix for Nonetype in image [Bridgetower] Fix for Nonetype in image Feb 14, 2024
@MohitIntel MohitIntel closed this Feb 16, 2024
@MohitIntel MohitIntel deleted the mdeopujari/SW-174850 branch February 16, 2024 04:39
astachowiczhabana added a commit that referenced this pull request Nov 28, 2024
…39)

Co-authored-by: Adam Stachowicz <105052242+astachowiczhabana@users.noreply.github.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.

4 participants