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

Color temperature is not proper #108

Open
daudarfin opened this issue May 9, 2024 · 8 comments
Open

Color temperature is not proper #108

daudarfin opened this issue May 9, 2024 · 8 comments

Comments

@daudarfin
Copy link

daudarfin commented May 9, 2024

Describe the bug
Color temperature is not proper when I capture using the github application on non-supported devices. I don't see this issue with the Play store application.

Play store version works perfectly fine without any issues on non-supported devices.

To Reproduce
Steps to reproduce the behavior:

  1. Capture an image in Photo mode
  2. See the color temperature

Expected behavior
Color temperature should be similar what we see in the camera preview

Screenshots

IMG_20240509_170446
IMG_20240509_170825

I could reproduce this issue on multiple android devices (including Google Pixel phones). Please note, my devices are not part of the supported list.

@eszdman
Copy link
Owner

eszdman commented May 9, 2024

Does this problem persist with the latest source version? Where you can choose a color correction method

@daudarfin
Copy link
Author

daudarfin commented May 10, 2024

I am using the latest source version (checked-out yesterday) and I don't see such method for color correction.

There is a color method (not correction) which I already tried but it doesn't make any differences.

Color method:

  1. Auto
  2. Characteristics
  3. Capture

Issue persists with all of the above methods.

Could you please let me know what is missing here when compared with Play store? Any changes (to fix this issue) required in source code would also help.

@eszdman
Copy link
Owner

eszdman commented May 10, 2024

Most likely this is due to the current session, I am using raw and jpeg stream at the same time
Some devices begin to behave differently when used this way at the moment I see the only solution is to enable dual session in device config file or upload config to repository

@daudarfin
Copy link
Author

daudarfin commented May 11, 2024

Dual session will fix this issue?

Could you please let me know how to enable this session in the source code? Is it JPEG + RAW option available inside Settings?

Is there any plan to fix this issue where device begin to behave differently?

@eszdman
Copy link
Owner

eszdman commented May 11, 2024

Could you please let me know how to enable this session in the source code?

File deviceSpecific = new File(sPHOTON_TUNING_DIR, "DeviceSpecific.txt");

Create DeviceSpecific.txt in DCIM/PhotonCamera/Tuning and write this line
isDualSessionSupported=true

@daudarfin
Copy link
Author

daudarfin commented May 13, 2024

Create DeviceSpecific.txt in DCIM/PhotonCamera/Tuning and write this line isDualSessionSupported=true

This doesn't help. Issue still persists even with dual session flag enabled.

After enabling, there is a crash observed while taking a picture.

java.lang.IllegalArgumentException: CaptureRequest contains unconfigured Input/Output Surface!

Even after fixing this crash, the color temperature is still improper.

@eszdman
Copy link
Owner

eszdman commented May 13, 2024

Fixed crash in latest versions

@daudarfin
Copy link
Author

Thanks. The crash is fixed in the latest version.

But the color temperature issue still persists. The dual session support also didn't work. Could you please help me fixing this issue?

IMG_20240514_105954
IMG_20240514_110136

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

2 participants