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

Make Single Camera Session for Both Preview/Capture #67

Conversation

NaveenVenturi1203
Copy link

Issue Detailed: Capture is creating a new camera session leading to restart of camera sensor in HAL, Due to the restart first few frames of camera are over exposed and when JPEG Encoded it is leading to yellow frames.

Issue Fixed: Made Single Camera Session to be used for both preview/capture which will avoid restart of camera sensor and the jpeg encoding is proper.

Tracked-On: OAM-126049

Issue Detailed: Capture is creating a new camera session leading to
restart of camera sensor in HAL, Due to the restart first few frames of
camera are over exposed and when JPEG Encoded it is leading to yellow
frames.

Issue Fixed: Made Single Camera Session to be used for both
preview/capture which will avoid restart of camera sensor and the jpeg
encoding is proper.

Tracked-On: OAM-126049
Signed-off-by: NaveenVenturi1203 <[email protected]>
Copy link

@balajim001 balajim001 left a comment

Choose a reason for hiding this comment

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

LGTM

@sysopenci sysopenci added PR approved and removed Pending PR Review Pending PR Review labels Nov 27, 2024
@sysopenci sysopenci added Developer Approved and removed Pending Developer Approval Pending Developer Approval labels Nov 27, 2024
@sysopenci
Copy link

Android CI has started Engineering Build for this issue ,Please check the linked Tracked-On issue/Android CI Web for more details.

@sysopenci
Copy link

SUCCESS: Android CI has completed Engineering Build for this issue.Please check the linked Tracked-On issue/Android CI Web for more details.

@sysopenci sysopenci added Engineering Build Successful Engineering Build Successful and removed Engineering Build Not Started Engineering Build Not Started labels Nov 27, 2024
@sysopenci
Copy link

Android CI has started MERGE Build for this pr ,Please check the linked Tracked-On issue/Android CI Web for more details.

1 similar comment
@sysopenci
Copy link

Android CI has started MERGE Build for this pr ,Please check the linked Tracked-On issue/Android CI Web for more details.

@sysopenci
Copy link

Android CI has completed MERGE Build for this pr, build is FAILURE. Please check the linked Tracked-On issue/Android CI Web for more details.

@sysopenci sysopenci merged commit cee8916 into projectceladon:celadon/u/mr0/master Nov 28, 2024
19 checks passed
@sysopenci
Copy link

Android CI has completed MERGE Build for this pr, build is SUCCESS. Please check the linked Tracked-On issue/Android CI Web for more details. For Binaries: /cactus-absp-or-local/celadon_umr0_master-merge/276

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

Successfully merging this pull request may close these issues.

3 participants