dji-sdk / Mobile-SDK-Android-V5

MSDK V5 Sample
Other
247 stars 126 forks source link

Set White balance and ISO - H20T #375

Open antonymarion opened 1 week ago

antonymarion commented 1 week ago

Hi Dji,

On 300 RTK H20T

KeyManager
                                        .getInstance()
                                        .setValue(createCameraKey(CameraKey.KeyWhiteBalance, cameraIndex, lensType),
                                        whiteBalanceinfo, object : CompletionCallback {
                                            override fun onSuccess() {
                                                Log.i("LOG", "Set white Balance success")
                                            }

                                            override fun onFailure(error: IDJIError) {
                                                Log.e("LOG- ERROR", error.toString())
                                            }
                                        });

Using the code below I am getting:

errorImp{errorType='CORE', errorCode='REQUEST_HANDLER_NOT_FOUND', innerCode='', description='null', hint='error code = -1'}

antonymarion commented 1 week ago

To set the ISO with

KeyManager.getInstance().setValue(
                                            KeyTools.createCameraKey(CameraKey.KeyISO, cameraIndex, lensType),
                                            targetIso, callbackFn (like above)

I am getting CANNOT_START_TASK_VLOTAGE_ALARM

dji-dev commented 1 week ago

Agent comment from yating.liao in Zendesk ticket #110006:

The information I found indicates that the lenses in the H20 series do not have a white balance adjustment function. Can you adjust this in the official app?

The adjustment of ISO usually needs to be done in manual exposure mode, as mentioned in the KeyISO interface description. Did you set it in manual mode? Also, what is the lens displaying the current image on the H20T?

°°°

antonymarion commented 1 week ago

Can you adjust this in the official app?

=> Could you just please provide the information in the official Doc from DJI; for instance a list of compability will be appreciated here:

image

It would help developpers to access the right info at the right place (and avoid to navigate in ticket on Github to get the info)

regards

antonymarion commented 1 week ago

By the way can you confirm ISO is only working for ZOOM camera on RTK, normal?

antonymarion commented 1 week ago

And what is the limit of H20T for ISO:

image

I get the following range from SDK

"isoRange":["ISO_100","ISO_200","ISO_400","ISO_800","ISO_1600","ISO_3200","ISO_6400","ISO_12800"]

But I succeed to put "iso":"ISO_25600", ... normal?

For greater values then I get an error as expected.

image

dji-dev commented 1 week ago

Agent comment from yating.liao in Zendesk ticket #110006:

The maximum value provided by KeyISORange is the maximum value currently supported by the lens. Have you set the lens type to zoom lens when obtaining the range? If the lens type is set to default, the data retrieved may not be from a zoom lens.

°°°

antonymarion commented 1 week ago

Yes, but... why does it returns range values with other lenses if we cannot use ISO for other lenses than Zoom ?

dji-dev commented 4 days ago

Agent comment from yating.liao in Zendesk ticket #110006:

I assume that you have set the lensType to default when obtaining ["ISO_100","ISO_200","ISO_400","ISO_800","ISO_1600","ISO_3200","ISO_6400","ISO_12800"], which is actually a method we do not recommend for camera parameters. For cameras, I suggest setting the camera position and lens type unless this interface is unrelated to the lens type, such as the photo-taking interface. You may want to consider using the correct method of use, as incorrect usage may result in obtaining inaccurate data.

°°°

antonymarion commented 4 days ago

Could we consider instead that the SDK always returns correct values and does not return incorrect values ?

Incorrect SDK usage should not return misleading values.

If default camera should not be used for getting iso, then SDK should return an error with a descriptive message, for instance: "ISO not available for default camera, please switch to zoom"

dji-dev commented 2 days ago

Agent comment from yating.liao in Zendesk ticket #110006:

Yes, I will submit your suggestion to the team. Perhaps we should provide explanations for users in the API documentation.

°°°