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

changelog of trunk including comments / devtalk

  • 298 Replies
  • 182454 Views
*

Offline reyalp

  • ******
  • 13353
Re: changelog of trunk including comments / devtalk
« Reply #290 on: 25 / March / 2021, 00:19:33 »
Advertisements
I noticed in DryOS r54 and later, Rec2PB (used in dryos switch_mode_usb) takes an argument. From the code, the argument should be a levent ID, or -1. For disassembly, our existing switch_mode_usb code would generally end up passing 0, since the mode argument is in r0.

Firmware that expect the argument could be identified by the string "AC:Rec2PB x Repost", but rather than adding stuff to the sig finders and a new define, in trunk 5798 I just made it pass -1 all the time.

PB2Rec still does not take arguments on these cams.
Don't forget what the H stands for.

*

Offline reyalp

  • ******
  • 13353
Re: changelog of trunk including comments / devtalk
« Reply #291 on: 30 / June / 2021, 02:13:27 »
I checked in some fixes for the histogram in r5977. Long discussion with various tangents here

The main issues addressed are:
* Histogram would sample incorrect data when viewport width wasn't 720 or 960
* High res displays could overflow the histogram counter.

The changes are intended to be stay close to the old behavior (aside from the bugs) with intention of backporting it to the stable branch. I might try to do some additional optimization in the trunk after that's done.

Attached is some debugging code I used, for future reference. It times various stages parts of the process, displays viewport dimensions from CHDK and GetVRAM*PixelsSize, and the actual number of pixels sampled. It also allows drawing the sampled pixels on the viewport.

Don't forget what the H stands for.

*

Offline reyalp

  • ******
  • 13353
Re: changelog of trunk including comments / devtalk
« Reply #292 on: 04 / July / 2021, 02:49:13 »
In trunk r5981, I fixed some issues with the histogram used by script get_live_histo and custom auto iso:
* incorrect total for some viewport sizes
* ensure EVF and HDMI do not overflow ushort count
* ensure histogram data always initialized, make missing viewport buffer equivalent to black screen
Don't forget what the H stands for.

Re: changelog of trunk including comments / devtalk
« Reply #293 on: 04 / July / 2021, 03:34:57 »
@reyalp

Just downloaded r5981 on my M3 and noticed it's reverted to flashing all the time.

I thought you had incorporated the 'philmoz fixes'?

But I guess I wrong?


*

Offline reyalp

  • ******
  • 13353
Re: changelog of trunk including comments / devtalk
« Reply #294 on: 04 / July / 2021, 03:40:56 »
Just downloaded r5981 on my M3 and noticed it's reverted to flashing all the time.

I thought you had incorporated the 'philmoz fixes'?
The ximr drawing changes are not in the trunk.

The live histogram changes mentioned above probably aren't significant for you, but I did merge them into the ximr branch, so they'll be in philmoz's build whenever he updates.
Don't forget what the H stands for.

*

Offline philmoz

  • *****
  • 3332
    • Photos
Re: changelog of trunk including comments / devtalk
« Reply #295 on: 04 / July / 2021, 03:52:47 »
@pigeonhill - ximr builds updated on Google drive with latest changes.
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)

Re: changelog of trunk including comments / devtalk
« Reply #296 on: 04 / July / 2021, 03:54:19 »
Cheers  :)

*

Offline reyalp

  • ******
  • 13353
Re: changelog of trunk including comments / devtalk
« Reply #297 on: 05 / July / 2021, 00:20:46 »
Histogram fixes above and Lua fixes from https://chdk.setepontos.com/index.php?topic=650.msg146578#msg146578 are merged to the 1.5 branch as of build 5987
Don't forget what the H stands for.


*

Offline reyalp

  • ******
  • 13353
Re: changelog of trunk including comments / devtalk
« Reply #298 on: 31 / July / 2021, 20:31:50 »
In trunk 6003 I fixed shooting_get_user_av_id to return .id (the CHDK index / id) rather than prop_id (apex96) value. This appears to have gotten switched in r3035.

shooting_set_user_av_by_id correctly expected an id value. shooting_set_user_av_by_id_rel was broken, since it uses shooting_get_user_av_id to calculate the value for _set.

Despite changing long-standing behavior, I'll merge this to 1.5 as a bug fix (r6004), since the buggy behavior is unlikely to be useful.
« Last Edit: 31 / July / 2021, 21:10:04 by reyalp »
Don't forget what the H stands for.

 

Related Topics