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

✨ Expose Camera Intrinsic Parameters #3032

Open
2 tasks done
pvedula7 opened this issue Jun 28, 2024 · 2 comments
Open
2 tasks done

✨ Expose Camera Intrinsic Parameters #3032

pvedula7 opened this issue Jun 28, 2024 · 2 comments
Labels
✨ feature Proposes a new feature or enhancement

Comments

@pvedula7
Copy link

pvedula7 commented Jun 28, 2024

What feature or enhancement are you suggesting?

Exposing camera intrinsic parameters to JS. This thread: #2998 referenced grabbing calibration where @mrousavy said was lots of lines of code, but exposing only intrinsic should be easier since we do not need the access the calibration data classes.

Android exposes it in CameraCharacteristics.LENS_INTRINSIC_CALIBRATION here
IOS has an isCameraIntrinsicMatrixDeliveryEnabled on AVCaptureConnection and available in the CMBuffer cameraIntrinsicMatrix

Why?
Intrinsic parameters are necessary for any kind of 2D to 3D CV techniques such as point clouds. The are the most valuable part of calibration data as extrinsic parameters are affected by several other factors.

What Platforms whould this feature/enhancement affect?

iOS, Android

Alternatives/Workarounds

N/A at the moment

Additional information

@pvedula7 pvedula7 added the ✨ feature Proposes a new feature or enhancement label Jun 28, 2024
@maintenance-hans maintenance-hans bot changed the title Expose Camera Intrinsic Parameters✨ ✨ Expose Camera Intrinsic Parameters Jun 28, 2024
Copy link

maintenance-hans bot commented Jun 28, 2024

Guten Tag, Hans here.

Note

New features, bugfixes, updates and other improvements are all handled mostly by @mrousavy in his free time.
To support @mrousavy, please consider 💖 sponsoring him on GitHub 💖.
Sponsored issues will be prioritized.

@mrousavy
Copy link
Owner

Hey - good feature request.

As Hans said, I work on this in my free time and at the moment Camera intrinsic data delivery has a very low priority for me.

If you want this prioritized consider contacting me thru my consultancy agency, otherwise this stays open until I decide to tackle it in my free time in a few weeks or months or so.

I had a working prototype where I delivered the matrix thru the CMSampleBuffer, but it was a lot of (Objective-)C code that was required for this. On Android, it's also a bit different as it's not available per Frame, but on the device already and I don't want two different solutions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ feature Proposes a new feature or enhancement
Projects
None yet
Development

No branches or pull requests

2 participants