Skip to content

[android] fix yuv conversion - remove define#50951

Closed
IvanKobzarev wants to merge 1 commit intogh/ivankobzarev/21/basefrom
gh/ivankobzarev/21/head
Closed

[android] fix yuv conversion - remove define#50951
IvanKobzarev wants to merge 1 commit intogh/ivankobzarev/21/basefrom
gh/ivankobzarev/21/head

Conversation

@IvanKobzarev
Copy link
Copy Markdown
Contributor

@IvanKobzarev IvanKobzarev commented Jan 22, 2021

Stack from ghstack:

Differential Revision: D26021488

Explanation of the issue: https://discuss.pytorch.org/t/trouble-with-yuv420-to-float-tensor-conversion/106721/4

Prevent expansion of macro that does " - normMeanRm255" first - remove macro

@facebook-github-bot
Copy link
Copy Markdown
Contributor

facebook-github-bot commented Jan 22, 2021

💊 CI failures summary and remediations

As of commit 3d29ea3 (more details on the Dr. CI page):


💚 💚 Looks good so far! There are no failures yet. 💚 💚


This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.

Please report bugs/suggestions to the (internal) Dr. CI Users group.

IvanKobzarev added a commit that referenced this pull request Jan 22, 2021
ghstack-source-id: 5dc76c6
Pull Request resolved: #50951
@IvanKobzarev IvanKobzarev requested a review from fmassa January 22, 2021 18:07
Copy link
Copy Markdown
Member

@fmassa fmassa left a comment

Choose a reason for hiding this comment

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

Thanks!

@facebook-github-bot
Copy link
Copy Markdown
Contributor

@IvanKobzarev merged this pull request in c908ebd.

@facebook-github-bot facebook-github-bot deleted the gh/ivankobzarev/21/head branch January 26, 2021 15:21
laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 24, 2026
Summary: Pull Request resolved: pytorch#50951

Test Plan: Imported from OSS

Reviewed By: fmassa

Differential Revision: D26021488

Pulled By: IvanKobzarev

fbshipit-source-id: 6d295762bb1160a3ed8bafac08e03e1eeb07d688
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants