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

Very slow on older devices #3

Open
aayushkapoor206 opened this issue Oct 15, 2015 · 9 comments
Open

Very slow on older devices #3

aayushkapoor206 opened this issue Oct 15, 2015 · 9 comments
Assignees

Comments

@aayushkapoor206
Copy link

The frame rate is very low, around 0.5 fps on iPhone 4S and the app receives memory warning in about 8 seconds. Hope that helps! 👍

@D-Link13
Copy link

is there any options that can make this app more faster?

@ryanmasondavies
Copy link
Owner

@aayushkapoor206 @D-Link13 I'm looking into this to see if we can speed it up.

@ryanmasondavies ryanmasondavies self-assigned this Mar 25, 2016
@D-Link13
Copy link

I found out this one tutorial. It uses AVCaptureMetadataOutput() instead of CIDetector, and it works faster on the device. But of course it's without any drawing content, only detecting face.

@ryanmasondavies
Copy link
Owner

Thanks for the pointer! If that's a faster API for detecting faces, that would definitely help with performance. I'll look into it for use here. If Apple can detect faces in real-time in the Camera app, we should be able to do it too!

There are also a few conversions that stand out to me (primarily, conversion of CIImage to UIImage for display in ViewController) where we might gain a few minor performance improvements.

@D-Link13 would you be able to help with testing / implementation?

@ryanmasondavies ryanmasondavies modified the milestone: Real-time Performance Mar 25, 2016
@D-Link13
Copy link

Well, I have already done an app based on that subject (avcapturemetadata). In wich way it will be the best to provide it for you? I am not sure about the clearest of the code, but it works! So it will be good if you help me with possible refactor.

@ryanmasondavies
Copy link
Owner

You could either email it to me (see profile) or – preferably – clone this repo and add your code as a separate project (maybe name it AVCaptureMetadata). I'll happily accept a pull request. If you're interested in contributing further than that, I can also grant push access for you! And yeah, happy to help with refactoring. 👍

@ryanmasondavies
Copy link
Owner

To be clear, if you've written a whole app and don't want to share the entire project – I'm only interested in the use of AVCaptureMetadata, so you could just put the relevant file in a folder and push that.

@D-Link13
Copy link

D-Link13 commented Mar 31, 2016 via email

@ryanmasondavies
Copy link
Owner

@D-Link13 I think maybe you attached a file to an email reply, but that didn't get included in your GitHub comment?

Could you possibly make a pull request, push the repo to your own GitHub profile, or copy and paste the code directly into a comment on the website? Or if you'd prefer, you can email it to me directly at mail@ryanjdavies.com and I'll take a look.

I'll gladly help with improving the code once I can try it! Thanks! 😃

@ryanmasondavies ryanmasondavies removed this from the Real-time Performance milestone Oct 12, 2019
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

No branches or pull requests

3 participants