OpenGoPro icon indicating copy to clipboard operation
OpenGoPro copied to clipboard

Allow for zoom setting through Bluetooth API

Open EddyCelis opened this issue 3 years ago • 10 comments

Is your feature request related to a problem or use case? Please describe. It is now possible to get the zoom status (percent of zoom) through the API. There is no documented way to SET the current zoom on the camera.

Describe the solution you'd like Add a setting to allow for changing the zoom setting via Bluetooth

Describe alternatives you've considered None.

Additional context It seems the feature does not exist in the firmware, as the Quik app does not allow to change the zoom either.

EddyCelis avatar Apr 23 '22 20:04 EddyCelis

You can.

/gopro/camera/digital_zoom?percent=50 (from: https://gopro.github.io/OpenGoPro/http_2_0)

KonradIT avatar Apr 23 '22 20:04 KonradIT

Should have specified: through Bluetooth.

EddyCelis avatar Apr 24 '22 14:04 EddyCelis

Is it possible to set Zoom level through Bluetooth?

shubham9436 avatar Jun 13 '22 11:06 shubham9436

Most likely not, why would GoPro allow for zooming in over BLE when the app uses WiFi for preview, and thus, zooming is done over WiFi as well.

KonradIT avatar Jun 13 '22 11:06 KonradIT

No it is not.

I flagged it as an enhancement request.

From: Shubham Baranwal @.> Sent: Monday, June 13, 2022 7:07 AM To: gopro/OpenGoPro @.> Cc: EddyCelis @.>; Author @.> Subject: Re: [gopro/OpenGoPro] Allow for zoom setting through API. (Issue #164)

Is it possible to set Zoom level through Bluetooth?

— Reply to this email directly, view it on GitHub https://github.com/gopro/OpenGoPro/issues/164#issuecomment-1153780846 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AMDE2FBIRLBVHASA6CDLZODVO4I5HANCNFSM5UFFXPRA . You are receiving this because you authored the thread. https://github.com/notifications/beacon/AMDE2FD4UOBS6UDAPKKYFEDVO4I5HA5CNFSM5UFFXPRKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOITCUY3Q.gif Message ID: @.*** @.***> >

EddyCelis avatar Jun 14 '22 02:06 EddyCelis

Hello, I'm also wondering why you want this. Can you provide a use case or some reasoning on what value this brings? Why is not acceptable for this command to only be exposed via HTTP?

tcamise-gpsw avatar Jun 14 '22 20:06 tcamise-gpsw

In our application we allow for control of several cameras and every camera is reset to a pre-configured preset at app startup. We wanted to add the capability to also set the zoom independent of the preset zoom setting, so that when the app sets up the camera at startup, the zoom setting is also adjusted. If this is only possible through http, the wifi connection will have to be connected (and acknowledged by the user) for every camera that is initialized.

Hello, I'm also wondering why you want this. Can you provide a use case or some reasoning on what value this brings? Why is not acceptable for this command to only be exposed via HTTP?

-- Reply to this email directly or view it on GitHub: https://github.com/gopro/OpenGoPro/issues/164#issuecomment-1155690399 You are receiving this because you authored the thread.

Message ID: @.***>

EddyCelis avatar Jun 14 '22 21:06 EddyCelis

Since the actual effect of setting the zoom isn't visible to the user until, at minimum, WiFi is connected, could you just set the zoom to your desired default value immediately after the WiFI is first connected? And more generally, do the same for any settings that only are relevant when WiFi is connected?

I suppose you incur a non-zero startup delay to set these defaults but it is only for the first WiFi connection and is almost certainly not noticeable.

From the other perspective, it is certainly possible to expose this setting via BLE. AT this time, these types of HTTP-specific settings are not exposed via BLE so this potentially opens the floodgates to exposing all settings, etc via BLE. Perhaps this is actually a good thing. I will take it up for discussion internally but I wouldn't expect anything in the immediate future.

tcamise-gpsw avatar Jun 14 '22 21:06 tcamise-gpsw

This is for a specific application where the zoom setting will change depending on other settings (non-GoPro) in the application at startup and while running. We could have 2 or more cameras running at the same time that could all have a need to change the zoom. As this is a ‘moving’ kind of application (car, airplane, boat), we absolutely want to make sure there is no distraction (e.g. need to acknowledge the wifi) while running.

But it is not an absolute deal breaker for the app, just a nice-to-have, so we have commented out the code for that feature.

Is there a list with enhancement requests? We have some other nice-to-haves 😉

From: Tim Camise @.> Sent: Tuesday, June 14, 2022 5:51 PM To: gopro/OpenGoPro @.> Cc: EddyCelis @.>; Author @.> Subject: Re: [gopro/OpenGoPro] Allow for zoom setting through API. (Issue #164)

Since the actual effect of setting the zoom isn't visible to the user until, at minimum, WiFi is connected, could you just set the zoom to your desired default value immediately after the WiFI is first connected? And more generally, do the same for any settings that only are relevant when WiFi is connected?

I suppose you incur a non-zero startup delay to set these defaults but it is only for the first WiFi connection and is almost certainly not noticeable.

From the other perspective, it is certainly possible to expose this setting via BLE. AT this time, these types of HTTP-specific settings are not exposed via BLE so this potentially opens the floodgates to exposing all settings, etc via BLE. Perhaps this is actually a good thing. I will take it up for discussion internally but I wouldn't expect anything in the immediate future.

— Reply to this email directly, view it on GitHub https://github.com/gopro/OpenGoPro/issues/164#issuecomment-1155745456 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AMDE2FE2JFU3WP42MT2IVB3VPD5FDANCNFSM5UFFXPRA . You are receiving this because you authored the thread. https://github.com/notifications/beacon/AMDE2FEBUDL54QJFCENZNPDVPD5FDA5CNFSM5UFFXPRKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOITRUNMA.gif Message ID: @.*** @.***> >

EddyCelis avatar Jun 14 '22 22:06 EddyCelis