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

Fix issue with green tint in Merrill images #120

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Alex-Vasile
Copy link

The right "shielded" area is not really shielded and can't be used for calculating the black level. Removing this area completely fixes the green tint in the shadows. Closes #117.

Below is a screenshot of the shielded area from the _SDI8040.X3F from x3f_test_files while is a DP2M file. I'm showing the green channel as shown by RawDigger, the red and blue channel are similar (except the red is much noisier and the blue much less noisy).
x3f

(As an aside, that region is very conspicuous with its linear increasing exposure may be used for something).

@mmroden
Copy link
Contributor

mmroden commented May 22, 2023

Thanks so much for this work! Would we expect the test image(s) to change as a result of this addition? If so, can you please also make a PR against x3f_test_images with the new image(s) converted with this fix? That way, we can make sure make check continues to pass.

@mmroden
Copy link
Contributor

mmroden commented May 22, 2023

(as a side note, this is obviously an argument for a monorepo reorganization, but as you can see, we haven't really been active in this project for 6 years)

@Alex-Vasile
Copy link
Author

Thanks so much for this work! Would we expect the test image(s) to change as a result of this addition? If so, can you please also make a PR against x3f_test_images with the new image(s) converted with this fix? That way, we can make sure make check continues to pass.

I have tested visually:

  1. I need to see how to get the tests running on my machine.
  2. Visually, the DPXM files do not have enough dark regions to show the issue. I plan on adding a imagine with lots of black regions, as well as some blown highlights (separate issue, blow highlights currently aren't handled properly)

… is not really shielded and can't be used for calculating the black level
@Alex-Vasile Alex-Vasile force-pushed the fix_green_cast_merrill branch from 21577cc to 1880430 Compare November 14, 2023 03:14
@antonzuykov
Copy link

antonzuykov commented Aug 9, 2024

(as a side note, this is obviously an argument for a monorepo reorganization, but as you can see, we haven't really been active in this project for 6 years)

Is this something you are still interested in supporting?
I think there is a way to cancel out the green circular cast. Hasselblad has a way of dealing with it for specific lenses by taking an image of just a white piece of plastic in front of a lens, creating that as a "sample" to treat the color-cast. I can link the video if you are interested. I was wondering if we can add this feature to kalpanika. It will certainly make it more useable, since the color cast on merrills ruins it.
@mmroden

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

Successfully merging this pull request may close these issues.

Green tint in shadows (with DP Merrills) since 0.57
3 participants