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

[LENS] Add Sony SEL1635GM and SEL85F18GM #496

Merged
merged 2 commits into from
Dec 9, 2019

Conversation

cryptomilk
Copy link
Contributor

@cryptomilk cryptomilk commented Feb 21, 2019

Hi,

this adds too new Sony lenses to lensfun.

You can find all the data here:

https://xor.cryptomilk.org/lensfun/SEL1635GM_calibration.tar.xz
https://xor.cryptomilk.org/lensfun/SEL85F18GM_calibration.tar.xz

Those files include the plots for TCA and vignetting. The package includes plots and preview files for vignetting corrections.

https://gitlab.com/cryptomilk/lens_calibrate

@junkyardsparkle
Copy link
Contributor

Thanks! Before adding this, could you confirm that the vignetting correction doesn't do any weird over-correcting on real-world (or non-diffuser flat-field) images? There's some discussion about that here, for context. More likely to be an issue with the wider lens, probably.

@cryptomilk
Copy link
Contributor Author

I would argue it works just fine: https://pixelbook.org/#15392672842961

@junkyardsparkle
Copy link
Contributor

It very well might, but it doesn't hurt to check against some images specifically intended to show up any over-correction. Processed images aren't great for that, even if they're great pictures. :)

For example, this one shows what might be over-correction, or it might just be the lighting or something else you did in processing. Anyway, to be clear, I'm not questioning your competence, it's just that the diffuser method of obtaining calibration images fails in some cases (I'm still trying to get a better grasp of which cases and why), so I'm just trying to encourage some extra checking for the profiles that go into the database. It's quite certain that some of the profiles already there suffer from this, but there's no way to test without the lens in hand. There's a recent case of somebody observing this problem here.

@cryptomilk
Copy link
Contributor Author

Torsten wrote in his original tutorial, that you should direct a light to the ceiling and then take the pictures. The problem is that on the side where the light is located, the picture is brighter. You can then less correction on one side and possibly over-correction on the other side.

That's why I suggest in my tutorial [1] to shoot into the sky on a cloudy day. Then you get an evenly exposed shot. If the pictures are to bright you can user a paper to make it darker or in the other direction wait for a thinner cloud coverage.

If you have suggestions, add a comment to the article or modify it directly, the source code is on github.

[1] https://pixls.us/articles/create-lens-calibration-data-for-lensfun/

@cryptomilk
Copy link
Contributor Author

We could extend the calibration script to generate small thumbnails of the vignetting pictures and include them in the tarball generated at the end.

@junkyardsparkle
Copy link
Contributor

(Forgive me if this isn't clear, it's late here.) I'm still trying to find the best practical method for obtaining good vignetting images. In theory, a very flat, diffuse, featureless surface lit only by direct sunlight, at an angle enough to stand in front of it without casting a shadow, should be good for this, but there are probably things that can go wrong with that, like everything else.

Soon, when I have the time, I want to set up a large piece of white acrylic in a window of an otherwise unlit room, and take shots ranging from zero to ~1 meter distance, to find out what distance is actually needed to avoid the problems with the diffuser. After that I might have more useful suggestions.

For now the main thing is that whatever method is used, the results should be checked against real-world images (of sky or some plain surface) to avoid over-correction. Thumbnails of the calibration images might be good for sanity-checking the images used... I've seen some dirty camera sensors that have a lot of "features" at f/22. ;)

@cryptomilk
Copy link
Contributor Author

But in direct sunlight you will have a shadows as you can't point the camera directly in the sun, on a cloudy day you can ...

Maybe we should move this discussion about how to take vignetting images to the pixls.us forums?

@junkyardsparkle
Copy link
Contributor

I mean using sunlight in a similar way to shooting a color calibration target, but the target is larger and devoid of features.

Forum discussions tend to have a poor signal/noise ratio in my experience, but start a discussion there if you think it will be productive. I need to do more empirical investigating before I do more discussing than I've already linked to.

@cryptomilk
Copy link
Contributor Author

Any news about your tests and findings?

@cryptomilk
Copy link
Contributor Author

cryptomilk commented Mar 16, 2019

Vignetting corrections in darktable and probably other raw developers doesn't not work yet, as exiv2 doesn't provide a focal distance estimation for Sony yet. See Exiv2/exiv2#582

I've already have a branch for darktable to fix it, but need support in exiv2 first. Probably later this year ...

@junkyardsparkle
Copy link
Contributor

Vignetting corrections in darktable and probably other raw developers doesn't not work yet, as exiv2 doesn't provide a focal distance estimation for Sony yet. See Exiv2/exiv2#582

It's amazing how many headaches such a "simple" thing can involve, isn't it? So, then, we should hold off merging this in the meantime? Did you get a chance to check if the default infinity corrections do anything unpleasant under the various circumstances discussed?

I did try putting a large piece of white acrylic in a window of an otherwise dark room, hoping to find a distancedistance from the lens face where the vignetting became less over-emphasized, but the lens I was profiling didn't produce any meaningful results... I'll try with another lens when I get a chance... along with a couple of other "mad science" experiments I've been pondering.

The idea of just profiling at whatever focus distance produces the strongest vignetting keeps sounding better and better. ;)

@cryptomilk
Copy link
Contributor Author

Can this be merged?

@junkyardsparkle
Copy link
Contributor

I'll merge it, but can you harmonize the the model name format with the rest of the db first (basically just Fn.n > f/n.n)? Also, the Samyang change makes no sense, as far as I can tell. The model+lang tags don't function as aliases, they're just for adding a "pretty" name when needed... and case doesn't matter for matching anyway: https://github.com/lensfun/lensfun/blob/master/tools/calibration_webserver/calibration.rst#create-lensfun-data

@cryptomilk
Copy link
Contributor Author

Updated, I didn't know about the pretty naming. I will add it to the howto on pixls.us.

@junkyardsparkle junkyardsparkle merged commit 3cf5fc3 into lensfun:master Dec 9, 2019
@junkyardsparkle
Copy link
Contributor

The most common use for the localisation of names is for fixed-lens cameras, as seen in the "compact-*" db files.

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.

2 participants