Skip to content

Fix issue with green tint in Merrill images#120

Open
Alex-Vasile wants to merge 1 commit intoKalpanika:masterfrom
Alex-Vasile:fix_green_cast_merrill
Open

Fix issue with green tint in Merrill images#120
Alex-Vasile wants to merge 1 commit intoKalpanika:masterfrom
Alex-Vasile:fix_green_cast_merrill

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