dji-sdk / Mobile-SDK-Android-V5

MSDK V5 Sample
Other
273 stars 145 forks source link

SimulatorManager setWindSpeed interface not working #434

Open brien-crean opened 1 week ago

brien-crean commented 1 week ago

I noticed that there is an undocumented setWindSpeed interface in the SimulatorManager class. However it does not appear to work.

Do I just need to set it once like this after initializing the Simulator? This would be extremely useful for simulation!

SimulatorManager.getInstance().enableSimulator(initSettings, object : CommonCallbacks.CompletionCallback {
  override fun onSuccess() {
    SimulatorManager.getInstance().setWindSpeed(SimulatorWindInfo(10, 3, 0))
  }

  override fun onFailure(error: IDJIError) {}
})
dji-dev commented 1 week ago

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

The MSDK currently does not have the feature to set wind speed. Are you looking to test it on the Mini 3 series drones?

°°°

brien-crean commented 1 week ago

@dji-dev thanks for the reply! Yes I am using a Mini 3 and a Mini 3 Pro. In the screenshot below you can see that there is an interface in the ISimulatorManager class to set the wind speed for the simulator. I'm currently running SDK version 5.11.0-a1. Is it possible to use this to simulate wind?

Screenshot 2024-10-18 at 10 25 30 AM

Decompiled Class Screenshot 2024-10-18 at 10 26 54 AM

dji-dev commented 4 days ago

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

The current version is unable to simulate wind speed. The interface you see does exist, but it is actually non-functional.

°°°

brien-crean commented 4 days ago

@dji-dev thanks for letting me know! Is it possible to make it functional in a future version of SDK5? It would be extremely useful for simulation testing before real flight!

dji-dev commented 3 days ago

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

I will forward your request to the team. However, there likely won't be a version that simulates wind speed released in the near future. I hope you can understand.

°°°