changelog of trunk including comments / devtalk - page 18 - General Discussion and Assistance - CHDK Forum  

changelog of trunk including comments / devtalk

  • 288 Replies
  • 150572 Views
*

Offline srsa_4c

  • ******
  • 3940
Re: changelog of trunk including comments / devtalk
« Reply #170 on: 19 / January / 2014, 13:26:54 »
Advertisements
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.

*

Online philmoz

  • *****
  • 3116
    • Photos
Re: changelog of trunk including comments / devtalk
« Reply #171 on: 20 / January / 2014, 03:18:46 »
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.
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)

*

Offline srsa_4c

  • ******
  • 3940
Re: changelog of trunk including comments / devtalk
« Reply #172 on: 20 / January / 2014, 15:50:36 »
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.

*

Online philmoz

  • *****
  • 3116
    • Photos
Re: changelog of trunk including comments / devtalk
« Reply #173 on: 21 / January / 2014, 05:06:24 »
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.
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)


*

Offline reyalp

  • ******
  • 11998
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.
Don't forget what the H stands for.

*

Offline msl

  • *****
  • 1262
  • 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
CHDK-DE:  CHDK-DE links

Re: changelog of trunk including comments / devtalk
« Reply #176 on: 03 / May / 2014, 09:46:15 »
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?

Ported :   A1200    SD940   G10    Powershot N    G16

*

Offline msl

  • *****
  • 1262
  • 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
CHDK-DE:  CHDK-DE links


*

Offline reyalp

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

*

Offline reyalp

  • ******
  • 11998
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.
Don't forget what the H stands for.

 

Related Topics