App crashes in Object Tracking fragment, when "Auto Mode" switch is activated

[MohammedZ666]

Bug Description App crashes in the Object Tracking Fragment when the Auto Mode switch is activated.

To Reproduce Steps to reproduce the behavior:

1. Go to Object Tracking
2. Turn on the Auto Mode switch
3. App may immediately crash
4. Logcat:

   07-22 12:02:06.196 26472-26554/org.openbot E/AndroidRuntime: FATAL EXCEPTION: inference
   Process: org.openbot, PID: 26472
   android.view.ViewRootImpl$CalledFromWrongThreadException: Only the original thread that created a view hierarchy can touch its views.
       at android.view.ViewRootImpl.checkThread(ViewRootImpl.java:8203)
       at android.view.ViewRootImpl.invalidateChildInParent(ViewRootImpl.java:1309)
       at android.view.ViewGroup.invalidateChild(ViewGroup.java:5430)
       at android.view.View.invalidateInternal(View.java:13959)
       at android.view.View.invalidate(View.java:13923)
       at android.view.View.invalidate(View.java:13907)
       at android.widget.TextView.checkForRelayout(TextView.java:8373)
       at android.widget.TextView.setText(TextView.java:5001)
       at android.widget.TextView.setText(TextView.java:4828)
       at android.widget.TextView.setText(TextView.java:4803)
       at org.openbot.objectNav.ObjectNavFragment.handleDriveCommand(ObjectNavFragment.java:500)
       at org.openbot.objectNav.ObjectNavFragment.lambda$processFrame$20$ObjectNavFragment(ObjectNavFragment.java:481)
       at org.openbot.objectNav.-$$Lambda$ObjectNavFragment$0cdHF_u8fuQJw1ABfb-EY_bHL6g.run(lambda)
       at android.os.Handler.handleCallback(Handler.java:739)
       at android.os.Handler.dispatchMessage(Handler.java:95)
       at android.os.Looper.loop(Looper.java:148)
       at android.os.HandlerThread.run(HandlerThread.java:61)

Fix In ObjectNavFragment.handleDriveCommand the textview controlInfo is being updated from a child thread. It should be called from the main thread, in this way:

  protected void handleDriveCommand(Control control) {
    vehicle.setControl(control);
    float left = vehicle.getLeftSpeed();
    float right = vehicle.getRightSpeed();
    requireActivity()
        .runOnUiThread(
            () ->
                binding.controllerContainer.controlInfo.setText(
                    String.format(Locale.US, "%.0f,%.0f", left, right)));
  }

In the same way, the onConnectionEstablished is updating another text view:

@Override
  public void onConnectionEstablished(String ipAddress) {
    requireActivity().runOnUiThread(() -> binding.ipAddress.setText(ipAddress));
  }

This is similar to this issue

Screenshots Here is the gif for a better explanation:

Smartphone (please complete the following information):

• Device: [e.g. Samsung J7]
• OS: [e.g. Android 6]

[thias15]

Interesting. In newer versions of Android this is not a problem.