Improving CHDK ND filter support

koshy

13 more from the IXUS family, should be all on the original list.
Didn't hunt for any ones I didn't have when I made that list.
Will do that after completing all on it...

srsa_4c
The very first few DIGIC II models don't have a full-blown capt_seq task and don't have all the hooks.

Turned out to be Ixus 30,40,50.

13 remaining Ixus.7z

9.18KB

koshy

Turns out I had that wrong... Ixus aren't done yet...
I have misplaced my 105,130,200,220,300,750,1000 still looking for 'em.
Found my 240, so here that is.

ndtest_ixus240_elph320hs-100d-1.5.0-5236-full_ALPHA.csv

4.64KB

koshy

Found 'em...

The Ixus 1000 is trouble.
That port has been trouble in multiple ways if I recall that right, what didn't we fuss with that one camera.

ndtest_ixus1000_sd4500-100d-1.5.0-5235-full.csv

2.41KB

koshy

Ok, this does it for the day. The Ixus cameras on my list are now really all touched once.
Ixus 200 and TX1 do not have ND.

@reyalp this was the list to cross complete ones off of I think: https://chdk.setepontos.com/d/13228/110

7 more Ixus.7z

4.79KB

reyalp

Thanks 😁

It'll take me a while to work through those. In the meantime, here's a test build for ixus970_sd890-100c.

It appears the PutInNDFilter and PutOutNDFilter stubs were reversed. This build uses the sig finder values, which are actually a different set of functions but probably equivalent.

ixus970_sd890-100c-1.5.0-5236-nd-test-reyalp-1.zip

830.97KB

koshy

reyalp
In the meantime, here's a test build for ixus970_sd890-100c.

Test result attached.

ndtest_ixus970_sd890-100c-1.5.0-5236-nd-test-reyalp-1.csv

5.37KB

reyalp

koshy

reyalp
In the meantime, here's a test build for ixus970_sd890-100c.

Test result attached.

Thanks, that one appears to work fine now, changes checked in for 1.4 and 1.5.

koshy

reyalp
Thanks, that one appears to work fine now, changes checked in for 1.4 and 1.5.

Nice, so this is actually useful. I'll go through the 33 A-Series cameras as I find the time and you'll just request any re-testing of the others as needed. I'll have to force myself to actually do the A-Series cameras asap I guess...

reyalp

koshy
Nice, so this is actually useful.

Yes, definitely. It's good for finding bugs like this, and the data from the successful will be a big help too.

I'll go through the 33 A-Series cameras as I find the time and you'll just request any re-testing of the others as needed. I'll have to force myself to actually do the A-Series cameras asap I guess...

No rush 😉

It looks like ixus85_sd770 had the same problem as ixus970_sd890, test build attached.

ixus85_sd770-100a-1.5.0-5240-reyalp-ndtest-1.zip

830.28KB

koshy

reyalp
No rush 😉

Recall what happend last time you said "No rush", there was a good but sad reason for that though, life got in the way.

reyalp

koshy
Recall what happend last time you said "No rush", there was a good but sad reason for that though, life got in the way.

Hence the smiley... but it is genuinely OK. Things happen when they happen.

I've gone through the logs from your last set. Most are fine, there are a few ports that seem to have issues but I'll need to dig a bit further to figure out the next steps.

Since the original post with the list seems to have issues, here's the A series list. There's no need to test other A series which are not defined to have an ND filter in CHDK

a1000
a1100
a1200
a1300
a1400
a2000
a2100
a2200
a2300
a2400
a2500
a2600
a3000
a3100
~~a3200~~
a3300
~~a3400~~
a4000
~~a410~~
a420
a430
a450
a460
a470
a480
~~a490~~
a495
a530
a550
a560
a580
a800
a810

reyalp

I updated the test script to better handle some of the issues seen in previous results. No need to re-test anything, but please use the new version (1.7) for future tests https://app.assembla.com/spaces/chdk/subversion/source/HEAD/trunk/CHDK/SCRIPTS/TEST/ndtest.lua

koshy

reyalp
It looks like ixus85_sd770 had the same problem as ixus970_sd890, test build attached.

Test attached.

reyalp
I updated the test script to better handle some of the issues seen in previous results. No need to re-test anything, but please use the new version (1.7) for future tests https://app.assembla.com/spaces/chdk/subversion/source/HEAD/trunk/CHDK/SCRIPTS/TEST/ndtest.lua

Thanks. I put it in place.

ndtest_ixus85_sd770-100a-1.5.0-5240-reyalp-ndtest-1.csv

5.35KB

koshy

reyalp
There's no need to test other A series which are not defined to have an ND filter in CHDK

I think A540, A570, A590 should be looked at in terms of "Is that so?" Probably A710 and A720, too.
For the first three I'd be extremely surprised if there was no ND filter and for the latter two still very surprised...
What to do to find out?

blackhole

koshy

reyalp
There's no need to test other A series which are not defined to have an ND filter in CHDK

I think A540, A570, A590 should be looked at in terms of "Is that so?" Probably A710 and A720, too.
For the first three I'd be extremely surprised if there was no ND filter and for the latter two still very surprised...
What to do to find out?

I've had the a590 for many years and there's no way that camera has an ND filter.
A quick look at the firmware shows that it contains only IrisActuator.c

koshy

blackhole
I've had the a590 for many years and there's no way that camera has an ND filter.
A quick look at the firmware shows that it contains only IrisActuator.c

O.k. did you check for actual DOF differences with different iris settings?
I would have expected these lens units to be the same:

koshy

koshy
I think A540, A570, A590 should be looked at in terms of "Is that so?" Probably A710 and A720, too.

O.k. always bet on stupidity. Was thrown off by form factor didn't look closely. All 5 have TV / AV modes.
@blackhole did the test I asked about so that's that...

reyalp

koshy

reyalp
It looks like ixus85_sd770 had the same problem as ixus970_sd890, test build attached.

Test attached.

Thanks. Confirmed fixed and checked in 😀

koshy
I think A540, A570, A590 should be looked at in terms of "Is that so?" Probably A710 and A720, too.

I'm very confident those are iris only. They don't have the ND related firmware tasks, or ND state/value functions, and they all do clearly have an adjustable aperture (they all have Av mode with multiple values, and Av overrides have been extensively used on these cameras)

Some other apparently iris-only cameras (A650, SX10, SX20 for example) do have some of the ND related code, but the ones tested earlier in the thread clearly adjusted the aperture when the ND functions were called.

koshy

So, after finding that everyone but me was alert today when it comes to irises, here is the 23 A cameras I have that needed ND tests. That means we're finally through the lot once. Specific tests hereafter...

23 As.7z

15.98KB

reyalp

koshy
So, after finding that everyone but me was alert today when it comes to irises, here is the 23 A cameras I have that needed ND tests. That means we're finally through the lot once. Specific tests hereafter...

Thanks. This is great test coverage, and we can confidently say the ND features work on most cameras. The firmware ND value is also generally in good agreement with the measured value.

One group of cameras appears to have issues:
A490, A495, A3100, ixus130_sd400, ixus300_sd4000
All of these are DryOS R43, propset 3 cams and Tv and Av prop override fail in "quick" mode.
ND control / value actually appears to work, but the failed override breaks the test. I haven't looked closely at the code for these yet.

The only other DryOS R43 cams with ND filter are untested: A3000, ixus105_sd1300.

Other failed cameras are the early vxworks cams without the remote hook, and a530 which should be new enough to have the have the hook but doesn't have it implemented.

« Previous Page Next Page »