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

Update ImageClassifier.java #27098

Closed
wants to merge 1 commit into from
Closed

Conversation

GuitarZhang
Copy link

When switching from GPU mode to CPU mode, gpuDelegate is not released. If switching to GPU again, gpuDelegate will be regenerated. If you keep switching, the memory will explode.

When switching from GPU mode to CPU mode, gpuDelegate is not released. If switching to GPU again, gpuDelegate will be regenerated. If you keep switching, the memory will explode.
@googlebot
Copy link

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 (e.g. I signed it!) and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

ℹ️ Googlers: Go here for more info.

@GuitarZhang
Copy link
Author

I signed it.

@gshashank84
Copy link
Contributor

I signed it.

There is ! missing in your reply.

@GuitarZhang
Copy link
Author

I signed it!

@rthadur rthadur self-assigned this Mar 25, 2019
@rthadur rthadur added this to Assigned Reviewer in PR Queue via automation Mar 25, 2019
@rthadur
Copy link
Contributor

rthadur commented Mar 25, 2019

@GuitarZhang i cannot see the CLA , can you please check again

@GuitarZhang
Copy link
Author

@GuitarZhang i cannot see the CLA , can you please check again

The user name I filled in was 258303742@qq.com(my email) or guitarzhang. Now I fill in the correct user name GuitarZhang, can you Please see again.

@googlebot
Copy link

CLAs look good, thanks!

ℹ️ Googlers: Go here for more info.

@rthadur rthadur added the comp:lite TF Lite related issues label May 20, 2019
@rthadur
Copy link
Contributor

rthadur commented May 20, 2019

@GuitarZhang sorry for the delay , can you please resolve conflicts.

@rthadur rthadur added the size:S CL Change Size: Small label May 20, 2019
@rthadur rthadur requested a review from aselle June 26, 2019 22:08
@rthadur rthadur added the awaiting review Pull request awaiting review label Jun 26, 2019
@mellanox-github
Copy link

Can one of the admins verify this patch?

@rthadur
Copy link
Contributor

rthadur commented Sep 16, 2019

changes are pushed by this commit 343ba94 , closing this , thank you

@rthadur rthadur closed this Sep 16, 2019
PR Queue automation moved this from Assigned Reviewer to Closed/Rejected Sep 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting review Pull request awaiting review cla: yes comp:lite TF Lite related issues size:S CL Change Size: Small
Projects
PR Queue
  
Closed/Rejected
Development

Successfully merging this pull request may close these issues.

None yet

5 participants