-
Notifications
You must be signed in to change notification settings - Fork 6k
Conversation
It looks like this pull request may not have tests. Please make sure to add tests before merging. If you need an exemption to this rule, contact Hixie on the #hackers channel in Chat. If you are not sure if you need tests, consider this rule of thumb: the purpose of a test is to make sure someone doesn't accidentally revert the fix. Ask yourself, is there anything in your PR that you feel it is important we not accidentally revert back to how it was before your fix? Reviewers: Read the Tree Hygiene page and make sure this patch meets those guidelines before LGTMing. |
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed (or fixed any issues), please reply here with What to do if you already signed the CLAIndividual signers
Corporate signers
ℹ️ Googlers: Go here for more info. |
|
cc @cbracken |
@xiaowei-guan |
This an implementation for Samsung Tizen platfrom : |
I got that. But it isn't used by any of the first party platforms. So what's the point of upstreaming this interface then? 🤦♂️ |
@jnschulze Although this new interface might not be used by the first-party platforms at the moment, it would greatly help custom embedder authors implement external textures on their platforms in a clean manner using standard desktop library headers. @cbracken Do you have any thoughts on this kind of matter? Can an interface (tentatively) only for custom embedders can be merged into the main stream by any chance? |
Hi, I just wanted to say that I'm writing a NVR (Network Video Recorder) in flutter and thus I need to not only render lots of video in flutter textures (thus use GPU rendering), but also pass the videos from the cameras in YUV form, each to its own texture, and then render them into the flutter texture. That is, do the YUV->RGB conversion in shader. Will this be possible with this PR? I'm not an OpenGL expert so I don't know if mixing Y,U,V textures into another one would work with this flutter texture here. I only mixed them together into the screen's buffer, or something like that. |
This pach only provides interface which privide a way to pass gpu buffer pointer, But the specific implementation needs to be completed by the platform itself. |
It looks like this pull request may not have tests. Please make sure to add tests before merging. If you need an exemption to this rule, contact Hixie on the #hackers channel in Chat. If you are not sure if you need tests, consider this rule of thumb: the purpose of a test is to make sure someone doesn't accidentally revert the fix. Ask yourself, is there anything in your PR that you feel it is important we not accidentally revert back to how it was before your fix? Reviewers: Read the Tree Hygiene page and make sure this patch meets those guidelines before LGTMing. |
b1a8771
to
2962099
Compare
Pinging @cbracken (Desktop Triage) |
This pull request executed golden file tests, but it has not been updated in a while (20+ days). Test results from Gold expire after as many days, so this pull request will need to be updated with a fresh commit in order to get results from Gold. |
Any update on this? |
I‘m going to close this given that #26840 provides a similar interface (which is effectively a superset of your proposal). |
This adds for supporting gpu buffer texture. Common API(shell/platform/common) only support pixel buffer texture, it need support gpu buffer texture to improve performance.
The related issue : flutter/flutter/issues/83693
If you had to change anything in the flutter/tests repo, include a link to the migration guide as per the breaking change policy.
Pre-launch Checklist
writing and running engine tests.
///
).If you need help, consider asking for advice on the #hackers-new channel on Discord.