Powershot A3400 IS - Porting thread - page 6 - General Discussion and Assistance

waterwingz

12088

Re: Powershot A3400 IS - Porting thread

« Reply #50 on: 05 / March / 2013, 08:44:24 »

Advertisements

Quote from: Breeksta on 04 / March / 2013, 23:47:42

However, I am using 101a vers. and it seems the scripts have not yet been polished.

AFAIK - the port is complete and as "polished" as it is going to get unless someone does detailed testing and reports back what they have found. "Scripts" have nothing to do with it.

Logged

Ported : A1200 SD940 G10 Powershot N G16

reyalp

13387

Re: Powershot A3400 IS - Porting thread

« Reply #51 on: 11 / May / 2013, 21:38:15 »

Some strange code in this port (re this thread: http://chdk.setepontos.com/index.php?topic=9804.msg100454#msg100454 )

If the camera has variable aspect ratios that affect the viewport layout, then it should have the actual values in these lookups. Based on the canon specs page, I'd guess it doesn't.

If the camera does not have variable aspect, then these functions should just return the correct value. In some cases, these are already the default values in generic/wrappers.c

in platform/a3400/lib.c

Code: [Select]

int vid_get_viewport_width()
{
    // viewport width table for each image size
    // 0 = 4:3, 1 = 16:9, 2 = 3:2, 3 = 1:1
    static long vp_w[4] = { 360, 360, 360, 360 };
    return vp_w[shooting_get_prop(PROPCASE_ASPECT_RATIO)];
}

long vid_get_viewport_height()
{
    // viewport height table for each image size
    // 0 = 4:3, 1 = 16:9, 2 = 3:2, 3 = 1:1
    static long vp_h[4] = { 240, 240, 240, 240 };
    return vp_h[shooting_get_prop(PROPCASE_ASPECT_RATIO)];
}

int vid_get_viewport_xoffset() 
{
    // viewport width offset table for each image size
    // 0 = 4:3, 1 = 16:9, 2 = 3:2, 3 = 1:1
    static long vp_w[4] = { 0, 0, 0, 0 };               // should all be even values for edge overlay
    return vp_w[shooting_get_prop(PROPCASE_ASPECT_RATIO)];
}

int vid_get_viewport_yoffset() 
{
    // viewport height offset table for each image size
    // 0 = 4:3, 1 = 16:9, 2 = 3:2, 3 = 1:1
    static long vp_h[4] = { 0, 0, 0, 0 };
    return vp_h[shooting_get_prop(PROPCASE_ASPECT_RATIO)];
}

Logged

Don't forget what the H stands for.

srsa_4c

4426

Re: Powershot A3400 IS - Porting thread

« Reply #52 on: 12 / January / 2014, 14:02:17 »

I have made some improvements to this port in changeset 3306
Main changes:
- DNG active area now includes all usable pixels, including the dark border which is only visible on wide angle images.
- the 'video' button can be used as ALT button
- the 'help' button is now the RAW toggle / debug shortcut button

« Last Edit: 12 / January / 2014, 14:09:06 by srsa_4c »

Logged

Menu jumping/flickering on M10?

arsenix

Re: Powershot A3400 IS - Porting thread

« Reply #53 on: 20 / June / 2015, 23:39:46 »

It seems like this may be a fairly low use port.

Any reason this build doesn't support remote shooting? Is this typically a hardware limitation or can I dig into the code to enable it?

Logged

waterwingz

12088

Re: Powershot A3400 IS - Porting thread

« Reply #54 on: 21 / June / 2015, 00:02:26 »

Quote from: arsenix on 20 / June / 2015, 23:39:46

It seems like this may be a fairly low use port.

It is not the only port that qualifies for that definition.

Quote

Any reason this build doesn't support remote shooting? Is this typically a hardware limitation or can I dig into the code to enable it?

USB remote shooting? Or remote PTP access? Before I look a the code I'm wondering why you think it's not supported?

Logged

Ported : A1200 SD940 G10 Powershot N G16

srsa_4c

4426

Re: Powershot A3400 IS - Porting thread

« Reply #55 on: 21 / June / 2015, 09:02:01 »

Quote from: arsenix on 20 / June / 2015, 23:39:46

Any reason this build doesn't support remote shooting?

If you mean CHDKPTP's rs -jpg command (aka filewritetask support), the reason is that nobody has requested it. I expect the support code (in code_gen.txt) to be the same as in a810/a1300 and the rest from that generation (a2300, a2400, a4000).

edit:
~~Wait a second. FWT on this camera is supported in trunk (CHDK 1.4).~~
edit2:
No, it's just local files in one of my working copies.
edit3:
untested patch attached