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

Sony A6300 ColorChecker shots #5580

Closed
Entropy512 opened this issue Dec 22, 2019 · 7 comments
Closed

Sony A6300 ColorChecker shots #5580

Entropy512 opened this issue Dec 22, 2019 · 7 comments
Labels
scope: color management Related to handling of colour processing scope: file format Camera or image file formats type: enhancement Something could be better than it currently is
Milestone

Comments

@Entropy512
Copy link
Contributor

D50 and stdA at https://filebin.net/ru7dcujz5eaoj00z

I will add a DCP later today, although as I understand it, @Beep6581 prefers to do the profiling runs himself, but I wouldn't mind some input on my own attempts at profiling. :)

@Beep6581
Copy link
Owner

Hi @Entropy512 , could you re-upload the files and ping me?

@Beep6581 Beep6581 added scope: color management Related to handling of colour processing type: enhancement Something could be better than it currently is scope: file format Camera or image file formats labels Jan 13, 2020
@Entropy512
Copy link
Contributor Author

https://filebin.net/dgjbravvlxmeh350 @Beep6581 here you are.

I'd be curious how the DCP I generated compares to yours.

Side note, you probably didn't see me ask this on pixls since you were on vacation, but I'm curious - what's the reasoning for passing -C for the D50 profile but not stdA in the instructions at https://rawpedia.rawtherapee.com/How_to_create_DCP_color_profiles#Creating_DCP_profiles_using_DCamProf ? Is that something you usually make a decision on a per-profile basis based on how dcamprof behaves when presented with particular data?

@Beep6581
Copy link
Owner

@Entropy512 there is no need for -C when using D50:
https://www.ludd.ltu.se/~torger/dcamprof.html#camera_cat

@Entropy512
Copy link
Contributor Author

Entropy512 commented Jan 17, 2020

OK, so then should that be removed from the command line shown on rawpedia?

If I'm reading that correctly, relighting in general should not be necessary for targets that have complete spectral data (which I believe we do have for the CC24/CCPassport? I can't recheck that until tomorrow.)

@Beep6581 Beep6581 added this to the v5.8 milestone Jan 18, 2020
@Beep6581
Copy link
Owner

@Entropy512 it's unnecessary in that example as far as I know, so it could be removed, but at the risk of forgetting to put it back in if you use a calibration illuminant different from D50.

@Entropy512
Copy link
Contributor Author

If I'm reading it correctly, then it looks like you don't need it even if the calibration illuminant is different, IF the calibration target you are using has spectral data instead of just XYZ data for each patch - which is the case at least for the .cie files in dcamprof's data-examples directory?

@Beep6581
Copy link
Owner

Yes, that seems to be the case for data from dcamprof, though the data from ArgyllCMS does not contain spectral data as far as I can tell.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope: color management Related to handling of colour processing scope: file format Camera or image file formats type: enhancement Something could be better than it currently is
Projects
None yet
Development

No branches or pull requests

2 participants