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.

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

[Exynos/Mali] ExternalTextureConverter issue with hand tracking sample App #41

Closed
cheongjunyoung opened this issue Aug 22, 2019 · 11 comments
Assignees
Labels
gpu MediaPipe GPU related issues platform:android Issues with Android as Platform type:external issue Issues in external dependencies

Comments

@cheongjunyoung
Copy link

At the start of hand tracking sample App, android runtime error(ExternalTextureConverter) occurs.

I tested the app using the galaxy s8.

is there any hardware requirements for running the hand tracking ??

2019-08-22 19:43:01.355 3675-3708/? E/AndroidRuntime: FATAL EXCEPTION: ExternalTextureConverter
Process: com.google.mediapipe.apps.handtrackinggpu, PID: 3675
java.lang.RuntimeException: glViewport: GL error: 0x505
at com.google.mediapipe.glutil.ShaderUtil.checkGlError(ShaderUtil.java:175)
at com.google.mediapipe.glutil.GlThread.bindFramebuffer(GlThread.java:116)
at com.google.mediapipe.components.ExternalTextureConverter$RenderThread.updateOutputFrame(ExternalTextureConverter.java:331)
at com.google.mediapipe.components.ExternalTextureConverter$RenderThread.renderNext(ExternalTextureConverter.java:246)
at com.google.mediapipe.components.ExternalTextureConverter$RenderThread.lambda$onFrameAvailable$0$ExternalTextureConverter$RenderThread(ExternalTextureConverter.java:210)
at com.google.mediapipe.components.ExternalTextureConverter$RenderThread$$Lambda$0.run(Unknown Source:4)
at android.os.Handler.handleCallback(Handler.java:873)
at android.os.Handler.dispatchMessage(Handler.java:99)
at android.os.Looper.loop(Looper.java:214)
at com.google.mediapipe.glutil.GlThread.run(GlThread.java:134)

@mcclanahoochie
Copy link

Hi,
I was able to get two galaxy s8 phones for testing, one with Adreno GPU and one with Mali GPU. I can reproduce this issue on the Mali chipset, while the Adreno works just fine.

It appears there is some hardware/memory limitations on the galaxy s8 G950FD with Mali chipset.
On that phone, none of the GPU examples with tflite seem to work as-is, but i was able to modify the CPU examples to run inference on the GPU (i.e. add "use_gpu: true" to the TfLiteInferenceCalculatorOptions in the graphs.

You may be able to create a CPU version of the hand graphs (remove all "_GPU" from stream names in the graphs) and run inference on the GPU still (add "use_gpu:true" to inference calculator options), as a workaround, but the performance won't be that good.

Meanwhile we will continue to improve support for a variety of devices.

@sevar83
Copy link

sevar83 commented Aug 24, 2019

Same on Galaxy S9.

@Macoron
Copy link

Macoron commented Aug 26, 2019

Reproduced on Galaxy S9, though Xiaomi Redmi 4x works fine

@leetenthu
Copy link

Same issue on Galaxy S9+ (SM-G965F/DS).

@mcclanahoochie mcclanahoochie changed the title ExternalTextureConverter issue with hand tracking sample App [Exynos/Mali] ExternalTextureConverter issue with hand tracking sample App Sep 12, 2019
@mgyong mgyong added the type:external issue Issues in external dependencies label Sep 18, 2019
@mgyong
Copy link

mgyong commented Sep 18, 2019

This bug is specific to TF. Will close in MediaPipe

@mgyong mgyong closed this as completed Sep 18, 2019
@leetenthu
Copy link

Is there a corresponding TF bug tracing this so that we can follow up?

@chuoling chuoling reopened this Sep 19, 2019
@chuoling
Copy link
Collaborator

We (MediaPipe) will follow up with TensorFlow team and provide an update here.

@hkawii
Copy link

hkawii commented Nov 19, 2019

Any updates on that issue ?

Same on S8+ and A20

@mgyong
Copy link

mgyong commented Nov 22, 2019

@hebamohamed @leetenthu @Macoron @sevar83 @cheongjunyoung Fixed bug in v0.6.5 release. Pls check out v0.6.5 and let us know if this release fixed the issue

@mgyong mgyong closed this as completed Nov 22, 2019
@VRShard
Copy link

VRShard commented Nov 23, 2019

Tested on Mali G51 multihandtrackinggpu --3D=true, on my side it is working now.

@leetenthu
Copy link

Tested on Galaxy S9+, it is working now, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
gpu MediaPipe GPU related issues platform:android Issues with Android as Platform type:external issue Issues in external dependencies
Projects
None yet
Development

No branches or pull requests

10 participants