sergiomb2 / ufraw

This is **not** the official ufraw, just why not migrate ufraw from cvs to git ?
https://sourceforge.net/p/ufraw/mailman/message/35450991/
GNU General Public License v2.0
16 stars 8 forks source link

Latest Canon Camer support #5

Closed maxbini closed 3 years ago

maxbini commented 5 years ago

Hi

this ufraw version is missing latest Canon cameras RAW compatibility (pink overlay) Here's the patch to update it with informations take from official ufraw source code. Tested with Canon EOS 6D Mark II.

diff --git a/dcraw.cc b/dcraw.cc index f9eab19..1e41f02 100644 --- a/dcraw.cc +++ b/dcraw.cc @@ -7509,12 +7509,16 @@ void CLASS adobe_coeff (const char make, const char model) { 6188,-1341,-890,-7168,14489,2937,-2640,3228,8483 } }, { "Canon EOS 5DS", 0, 0x3c96, { 6250,-711,-808,-5153,12794,2636,-1249,2198,5610 } },

sergiomb2 commented 5 years ago

Hi, Can you fork this repo , clone your repo , do the patch , after we may do git commit . , git push , and in github click in create a pull request ?

maxbini commented 5 years ago

Hi sergio

Yes I can but, after further tests, I realized that the patch is still missing some color calibration. The pink issue vanishes, but color rendition is different than using dcraw directly. I would say pretty wrong. I saw there is a reference for camera WB in wb_presets.c, but I dont't know how to get the correct value for 6D Mark II so I can'test. I'm looking for a solution.

Note that:

sergiomb2 commented 5 years ago

This is not the official ufraw , I did this repo using cvstogit [1] , but seems it is the most updated ufraw sources since authors stop to updating it.

ufraw is based on dcraw and BTW https://sourceforge.net/p/ufraw/feature-requests/327/ :disappointed: so your tests makes sense .

I think apps are moving to libRaw [2] since dcraw is also IIRC inactive .

[1] https://sourceforge.net/p/ufraw/mailman/message/35450991/

[2] https://apps.fedoraproject.org/packages/LibRaw

sergiomb2 commented 5 years ago

Hi sergio

Yes I can but, after further tests, I realized that the patch is still missing some color calibration. The pink issue vanishes, but color rendition is different than using dcraw directly. I would say pretty wrong. I saw there is a reference for camera WB in wb_presets.c, but I dont't know how to get the correct value for 6D Mark II so I can'test. I'm looking for a solution.

Note that:

* original ufraw taken from cvs can't compile, at least on my Ubuntu Disco. I suppose it hasn't the problem.

* nufraw has the same problem

* all ufraw ubuntu packages has the same problem

* all ufraw-dependent software in ubuntu has the same problem: digikam, shotwell, various viewer.

* all dcraw-dependent software in ubuntu has no problem

can you try build ufraw with https://github.com/sergiomb2/ufraw/tree/devel_dcraw_9.28 ? , I just tested I could read my raw samples , if is good the rebase ufraw with dcraw-9.28 is done .

Thanks.

sergiomb2 commented 4 years ago

Everything is already committed on https://github.com/sergiomb2/ufraw/tree/devel_dcraw_9.28 but in the last chunk [1] I added ",0" in each line .

[1]

+    {  5298000,2400,1766,12,12,44, 2, 8,0x94,0,2,"Canon","PowerShot SD300" },
+    {  6553440,2664,1968, 4, 4,44, 4, 8,0x94,0,2,"Canon","PowerShot A460" },
+    {  6573120,2672,1968,12, 8,44, 0, 8,0x94,0,2,"Canon","PowerShot A610" },
+    {  6653280,2672,1992,10, 6,42, 2, 8,0x94,0,2,"Canon","PowerShot A530" },
+    {  7710960,2888,2136,44, 8, 4, 0, 8,0x94,0,2,"Canon","PowerShot S3 IS" },
+    {  9219600,3152,2340,36,12, 4, 0, 8,0x94,0,2,"Canon","PowerShot A620" },
+    {  9243240,3152,2346,12, 7,44,13, 8,0x49,0,2,"Canon","PowerShot A470" },
+    { 10341600,3336,2480, 6, 5,32, 3, 8,0x94,0,2,"Canon","PowerShot A720 IS" },
+    { 10383120,3344,2484,12, 6,44, 6, 8,0x94,0,2,"Canon","PowerShot A630" },
+    { 12945240,3736,2772,12, 6,52, 6, 8,0x94,0,2,"Canon","PowerShot A640" },
+    { 15636240,4104,3048,48,12,24,12, 8,0x94,0,2,"Canon","PowerShot A650" },
+    { 15467760,3720,2772, 6,12,30, 0, 8,0x94,0,2,"Canon","PowerShot SX110 IS" },
+    { 15534576,3728,2778,12, 9,44, 9, 8,0x94,0,2,"Canon","PowerShot SX120 IS" },
+    { 18653760,4080,3048,24,12,24,12, 8,0x94,0,2,"Canon","PowerShot SX20 IS" },
+    { 19131120,4168,3060,92,16, 4, 1, 8,0x94,0,2,"Canon","PowerShot SX220 HS" },
+    { 21936096,4464,3276,25,10,73,12, 8,0x16,0,2,"Canon","PowerShot SX30 IS" },
+    { 24724224,4704,3504, 8,16,56, 8, 8,0x94,0,2,"Canon","PowerShot A3300 IS" },
+    { 30858240,5248,3920, 8,16,56,16, 8,0x94,0,2,"Canon","IXUS 160" },
sergiomb2 commented 3 years ago

Closing , because seems all that I can do, was done . Please open a new issue or reopen this one , if not

Thank you.