shuffleboard icon indicating copy to clipboard operation
shuffleboard copied to clipboard

Command activation and deactivation buttons do not work properly.

Open Tom-ne opened this issue 2 years ago • 12 comments

Hello, we have multiple commands in Shuffleboard. We have encountered an isssue that the button the activates and deactivates the command, almost always does not work on the first try, almost every time we need to press the button more than one time.

To Reproduce
Steps to reproduce the behavior:

  1. add commands to shuffleboard
  2. try to run them

Desktop (please complete the following information): Version 2023.3.2 JRE: 17.0.5+8 -- Eclipse Adoptium Operating system: Windows 10 10.0 amd64

Tom-ne avatar Feb 21 '23 21:02 Tom-ne

Same issue on Mac M1

hollisms avatar Feb 25 '23 12:02 hollisms

Can you upload Shuffleboard's logs from times that this happened? Shuffleboard log files are at ~/Shuffleboard/.

Starlight220 avatar Mar 19 '23 11:03 Starlight220

I think it might be this, but I am not sure shuffleboard.2023-02-13-19.05.06.log

Tom-ne avatar Mar 19 '23 13:03 Tom-ne

From a quick glance there seem to be multiple exceptions in the log. Will investigate more in a bit.

Starlight220 avatar Mar 19 '23 15:03 Starlight220

@Tom-ne the log you uploaded is from Shuffleboard v2023.1.1. The common exception there was fixed in #759 (released in 2023.4.1). There's some more errors/warnings there (some problems with loading saved layouts and CameraServer properties), but doesn't seem to show anything obviously part of this issue.

Starlight220 avatar Mar 19 '23 16:03 Starlight220

I will check the log file on our computers again and try to find the log file

Tom-ne avatar Mar 19 '23 22:03 Tom-ne

shuffleboard.2023-02-21-10.19.13.log I think it might this, near the end of the file

Tom-ne avatar Mar 20 '23 15:03 Tom-ne

That's curious. Are you sure that the command is still being published to NT when this happens?

Starlight220 avatar Mar 20 '23 16:03 Starlight220

I dont know. It should

Tom-ne avatar Mar 21 '23 19:03 Tom-ne

Does this reproduce with the newly-releases 2023.4.3? We hope that https://github.com/wpilibsuite/shuffleboard/pull/778 fixed this.

Starlight220 avatar Mar 29 '23 14:03 Starlight220

I don't know, I need to check

Tom-ne avatar Apr 02 '23 09:04 Tom-ne

This should have been fixed by the fix for allwpilib#5323 but it should be checked. Will need to use allwpilib HEAD for the test. I am not able to do the test now but if someone can then this can be closed.

chauser avatar Jul 24 '23 18:07 chauser