changelog of trunk including comments / devtalk - page 18 - General Discussion and Assistance - CHDK Forum  
« previous next »
Pages: 1 ... 13 14 15 16 17 [18] 19 20 21 22 23 ... 30   Go Down

changelog of trunk including comments / devtalk

  • 299 Replies
  • 227168 Views
*

Offline srsa_4c

  • ******
  • 4451
Re: changelog of trunk including comments / devtalk
« Reply #170 on: 19 / January / 2014, 13:26:54 »
Advertisements
Quote from: philmoz on 18 / January / 2014, 17:24:00
Edit2: The only other cameras with this sort of custom override are the ixus115, ixus30, ixus40 and s80. Do you have any of these you can test a patch on?
I thought there was more. I have all these cams (after all, I'm responsible for the majority of these exceptions), so I'll take over from here.
Logged
*

Offline philmoz

  • *****
  • 3450
    • Photos
Re: changelog of trunk including comments / devtalk
« Reply #171 on: 20 / January / 2014, 03:18:46 »
Quote from: srsa_4c on 19 / January / 2014, 13:26:54
Quote from: philmoz on 18 / January / 2014, 17:24:00
Edit2: The only other cameras with this sort of custom override are the ixus115, ixus30, ixus40 and s80. Do you have any of these you can test a patch on?
I thought there was more. I have all these cams (after all, I'm responsible for the majority of these exceptions), so I'll take over from here.

I'd already worked on the changes - patch attached if you're interested.

Includes makefile update to correctly identify IGNORE when checking for stubs errors.
Also adds a #define to remove the need for the custom version of GetParameterData and SetParameterData.

Phil.
Logged
CHDK ports:
  sx30is (1.00c, 1.00h, 1.00l, 1.00n & 1.00p)
  g12 (1.00c, 1.00e, 1.00f & 1.00g)
  sx130is (1.01d & 1.01f)
  ixus310hs (1.00a & 1.01a)
  sx40hs (1.00d, 1.00g & 1.00i)
  g1x (1.00e, 1.00f & 1.00g)
  g5x (1.00c, 1.01a, 1.01b)
  g7x2 (1.01a, 1.01b, 1.10b)
*

Offline srsa_4c

  • ******
  • 4451
Re: changelog of trunk including comments / devtalk
« Reply #172 on: 20 / January / 2014, 15:50:36 »
Quote from: philmoz on 20 / January / 2014, 03:18:46
I'd already worked on the changes - patch attached if you're interested.

Includes makefile update to correctly identify IGNORE when checking for stubs errors.
Also adds a #define to remove the need for the custom version of GetParameterData and SetParameterData.
Thanks. I have changed a few things, see attachment.

- found out that the guy who originally did the ixus40 port actually found and (identically) reimplemented a function what appears to be _IsStrobeChargeCompleted() in the firmware (it's not an eventproc) - I have added a sigfinder signature to find it (only ixus 30/40 are affected)
- corrected _GetPropertyCase() for ixus30, 40 - note that the Set and Get property/param functions return "status", not the requested value - yes, I know that the return value is probably never checked in CHDK
- I decided to restrict _MoveFocusLensToDistance() in the s80 port to AFL and movie mode (see the notes why)

I have tested all affected cams.
Logged
*

Offline philmoz

  • *****
  • 3450
    • Photos
Re: changelog of trunk including comments / devtalk
« Reply #173 on: 21 / January / 2014, 05:06:24 »
Quote from: srsa_4c on 20 / January / 2014, 15:50:36
Quote from: philmoz on 20 / January / 2014, 03:18:46
I'd already worked on the changes - patch attached if you're interested.

Includes makefile update to correctly identify IGNORE when checking for stubs errors.
Also adds a #define to remove the need for the custom version of GetParameterData and SetParameterData.
Thanks. I have changed a few things, see attachment.

- found out that the guy who originally did the ixus40 port actually found and (identically) reimplemented a function what appears to be _IsStrobeChargeCompleted() in the firmware (it's not an eventproc) - I have added a sigfinder signature to find it (only ixus 30/40 are affected)
- corrected _GetPropertyCase() for ixus30, 40 - note that the Set and Get property/param functions return "status", not the requested value - yes, I know that the return value is probably never checked in CHDK
- I decided to restrict _MoveFocusLensToDistance() in the s80 port to AFL and movie mode (see the notes why)

I have tested all affected cams.

Added in revision 3325.

Phil.
Logged
CHDK ports:
  sx30is (1.00c, 1.00h, 1.00l, 1.00n & 1.00p)
  g12 (1.00c, 1.00e, 1.00f & 1.00g)
  sx130is (1.01d & 1.01f)
  ixus310hs (1.00a & 1.01a)
  sx40hs (1.00d, 1.00g & 1.00i)
  g1x (1.00e, 1.00f & 1.00g)
  g5x (1.00c, 1.01a, 1.01b)
  g7x2 (1.01a, 1.01b, 1.10b)
*

Online reyalp

  • ******
  • 14079
Re: changelog of trunk including comments / devtalk
« Reply #174 on: 06 / April / 2014, 17:22:18 »
In trunk changeset  3415 I added something that has been on my todo list for ages: An alternative to os.listdir that doesn't load the entire directory into memory at once.

Documentation added on the wiki: http://chdk.wikia.com/wiki/Lua#Added_features_2

llibtst.lua is updated to test this, and works on both my dryos and vxworks cameras.
Logged
Don't forget what the H stands for.
*

Offline msl

  • *****
  • 1280
  • A720 IS, SX220 HS 1.01a
    • CHDK-DE links
Re: changelog of trunk including comments / devtalk
« Reply #175 on: 03 / May / 2014, 08:07:55 »
Many of the ported cameras are for a long time in the alpha or beta status. This confused many of the CHDK user.

I think it would be better, cancel the alpha and beta status after one year, if there are no known major problems with the camera.

msl
Logged
CHDK-DE:  CHDK-DE links
*

Offline waterwingz

  • ******
  • 12138
    • Waterwingz
Re: changelog of trunk including comments / devtalk
« Reply #176 on: 03 / May / 2014, 09:46:15 »
Quote from: msl on 03 / May / 2014, 08:07:55
Many of the ported cameras are for a long time in the alpha or beta status. This confused many of the CHDK user.  I think it would be better, cancel the alpha and beta status after one year, if there are no known major problems with the camera.
I agree.  The terms "alpha" and "beta" indicate that software is being actively tested and is at different levels of completeness.  It is not meant to indicate the presence or absence of bugs.

I don't think anyone else will disagree too much with your suggestion for removing alpha/beta status after a while.  It's probably only a question of somebody being willing to take a few hours to edit camera_list.csv  (assuming they lookup release dates and scan the appropriate porting threads).

And then somebody to periodically update that file as the magic "one year" expires.

I'd even suggest "one year" being too long - maybe six months after the last development related post in its porting thread or one year after alpha/beta status was designated?

Volunteers?

Logged
Ported :   A1200    SD940   G10    Powershot N    G16
*

Offline msl

  • *****
  • 1280
  • A720 IS, SX220 HS 1.01a
    • CHDK-DE links
Re: changelog of trunk including comments / devtalk
« Reply #177 on: 04 / May / 2014, 07:30:40 »
Six months sounds good. I have started a cleanup in the S/SX series.

msl
Logged
CHDK-DE:  CHDK-DE links
*

Online reyalp

  • ******
  • 14079
Re: changelog of trunk including comments / devtalk
« Reply #178 on: 04 / May / 2014, 18:12:13 »
Quote from: msl on 04 / May / 2014, 07:30:40
Six months sounds good. I have started a cleanup in the S/SX series.
Just want to say I agree with this plan. :)
Logged
Don't forget what the H stands for.
*

Online reyalp

  • ******
  • 14079
Re: changelog of trunk including comments / devtalk
« Reply #179 on: 18 / August / 2014, 23:42:35 »
In trunk changeset 3585 I renamed enable_highspeed_usb to enable_remote_hp_timer and changed both enable_remote_hp_timer and usb_force_active return booleans in Lua rather than a numbers.

This will break existing scripts using these functions. Sorry for any inconvenience.
Logged
Don't forget what the H stands for.
Pages: 1 ... 13 14 15 16 17 [18] 19 20 21 22 23 ... 30   Go Up
« previous next »
 

Related Topics