Skip to content
This repository has been archived by the owner on Dec 29, 2022. It is now read-only.

Add some features to CameraView #218

Closed
wants to merge 5 commits into from
Closed

Add some features to CameraView #218

wants to merge 5 commits into from

Conversation

sibelius
Copy link

  • upgrade gradle to 4.1
  • upgrade to android 26
  • implement white balance
  • implement zoom
  • implement focusDepth
  • implement record

this is based on @expo work https://github.com/expo/expo/tree/master/android/cameraview

the idea is to consume this package on https://github.com/lwansbrough/react-native-camera (check this react-native-camera/react-native-camera#1031)

@googlebot
Copy link

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. 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, please reply here (e.g. I signed it!) and we'll verify. Thanks.


  • If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
  • If your company signed a CLA, they designated a Point of Contact who decides which employees are authorized to participate. You may need to contact the Point of Contact for your company and ask to be added to the group of authorized contributors. If you don't know who your Point of Contact is, direct the project maintainer to go/cla#troubleshoot.
  • In order to pass this check, please resolve this problem and have the pull request author add another comment and the bot will run again.

@sibelius
Copy link
Author

I've signed the CLA, I don't know why this does not worked

@googlebot
Copy link

CLAs look good, thanks!

@sibelius
Copy link
Author

@yaraki can you take a look on this any time soon?

@brentvatne
Copy link

brentvatne commented Dec 12, 2017

@sibelius - the code that you used here is BSD licensed: https://github.com/expo/expo/blob/master/LICENSE -- it requires attribution and retaining the copyright notice. I'm not sure how this interacts with Google's CLA here, it might invalidate this PR.

@sibelius
Copy link
Author

should a put the expo LICENSE on THIRD-PARTY-LICENSES?

@elementlo
Copy link

I want to know now how to get the preview data , I see there is only method recording video but no one callback the preview frame , some one can help me? @sibelius

@aalices
Copy link

aalices commented Jan 31, 2018

@elementlo you should use setScanning(true) on you CameraView and then onPreviewFrame should be invoked within a Callback you previously added.

@elementlo
Copy link

@aalices OK , I'll have a try first , thank you.

if (mIsScanning) {
mPreviewRequestBuilder.addTarget(mScanImageReader.getSurface());
}
mCamera.createCaptureSession(Arrays.asList(surface, mStillImageReader.getSurface(),
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is an output while executing this statement:

E/CameraCaptureSession: Session 0: Failed to create capture session; configuration failed

This log is print from the internal source code. I researched but not get any progress.
I tried createCaptureSession with a list of two surface bugs got over, but Then I can not use the both surface as output surface.

So things become bad: I can only use one imagereader of the two, capture it or preview it.

Do you have any idea?

@bipinvaylu
Copy link

Hi, When do we have this PR gets merged? I want to do Preview, Recording and Preview frame processing parallel. I hope this PR incorporating those things. For more details, Please find my use case here: https://stackoverflow.com/questions/54091648/camera-2-api-to-preview-record-and-extract-the-preview-frame-to-firebase-face-d

@sibelius sibelius closed this Jan 10, 2019
@bipinvaylu
Copy link

Hey @sibelius, Closed PR without merge, It would be a great help if you can share latest codebase link here. I can't able to find the entria:G branch here.

@sibelius
Copy link
Author

sibelius commented Jan 10, 2019

We moved our efforts to react native camera, the code and improvements are there

If you want to contribute ther it would be great

@bipinvaylu
Copy link

Okay, Thanks for the response, So now you are developing the camera stuff into React Native, not in Native Android. That's nice. Atm, I am looking for the Previewing, Recording and Frame processing at the same time. As like I mentioned in my SO question.

@sibelius
Copy link
Author

You can do that using ML Kit

@bipinvaylu
Copy link

Yup, But I am unable to extract the Image/Frame when recording is going on. With Preview only, I am getting Image/Frames. Any help/insight will be appreciated on this question: https://stackoverflow.com/questions/54091648/camera-2-api-to-preview-record-and-extract-the-preview-frame-to-firebase-face-d

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

Successfully merging this pull request may close these issues.

7 participants