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

could you add a onFrameData callback in CameraListener? #176

Closed
leaf918 opened this issue Sep 20, 2017 · 15 comments
Closed

could you add a onFrameData callback in CameraListener? #176

leaf918 opened this issue Sep 20, 2017 · 15 comments

Comments

@leaf918
Copy link

leaf918 commented Sep 20, 2017

the frame data is important for realtime frame data processing.
thanks.

@MrOnyszko
Copy link

I agree with that idea :)

@dwillmc
Copy link
Member

dwillmc commented Nov 1, 2017

This is in the works now and coming soon. I want to do a bit on top of just passing the frame though. I've been adding a lot of NDK stuff lately to speed up processing and I'd like to structure that to be usable by CameraKit users. Especially with the frame callbacks where performance is incredibly important.

What sorts of things do you need to do with the frames?

@sksar
Copy link

sksar commented Nov 1, 2017

@dwillmc
Possibly a support for a function where we could specify a frame processor function that gets each frame, and also the processed frame (maybe like color enhanced, filter effects) is returned to the camera View.....

@morejump
Copy link

in my case, i want to get each frame to use for JNI with opencv c++, and return preview!

@leaf918
Copy link
Author

leaf918 commented Nov 10, 2017

@dwillmc In my case ,I have a lot of neural networks for real-time image processing,such as AR,robot vision

@xymelon
Copy link

xymelon commented Nov 13, 2017

@dwillmc In my case, I need frame to use for real-time OCR.

@monatis
Copy link

monatis commented Nov 17, 2017

@dwillmc In my case, I'm trying to detect motion (i.e., page turns) and take a photo for an automatic scan app.

@benjaminbarbe
Copy link

benjaminbarbe commented Dec 12, 2017

Do you have any update on this? @dwillmc

@austinkettner
Copy link
Member

@xymelon @monatis @benjaminbarbe Yes we plan to have this included in 1.0.0 release aimed for Jan 1, 2018. We hope to open a PR and Snapshot builds for this release, this week so you can comment, test, etc!

@dwillmc May have further input too

@browningweb
Copy link

Do you have any update on this...? @dwillmc

@austinkettner
Copy link
Member

We are hoping to issue a 1.0.0 PR soon for testing and a lot of this is included. @browningweb

@edwardotis
Copy link

edwardotis commented Apr 20, 2018

@austinkettner Can you post a link here to the PR and/or feature docs?
Otherwise, shouldn't this issue remain in Open state?

@kunaldawn
Copy link

any updates on this?

@gigadeplex
Copy link

BUMP......? should we switch camera package from camerakit...?

@austinkettner
Copy link
Member

@eviansplash @kunaldawn @edwardotis Here is the 1.0.0 PR: #318 give it a try and let us know a bit more about your use case? We are adding in additional calls ahead of merging and can specifically address this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests