Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Need to clearly indicate failure when configuring speed FB #305

Open
ted-miller opened this issue Sep 9, 2024 · 4 comments
Open

Need to clearly indicate failure when configuring speed FB #305

ted-miller opened this issue Sep 9, 2024 · 4 comments
Labels
bug Something isn't working ros2:humble rv:0.1.3 Reported in MotoROS2 0.1.3 yrc1000

Comments

@ted-miller
Copy link
Collaborator

We had a user get into this scenario. But he reports that he never saw the alarm, 8001 Speed FB enabled, reboot now. [10]

#294 (comment)

This results in MR2 appearing to freeze at startup and communication dropping to the agent.

The only way that I know it occurred was by looking through the alarm history.

I too believe that I have seen this scenario, but I just brushed it off and assumed that I wasn't paying attention.

We need to try to reproduce with a "clean" robot controller that has FSU enabled.

@ted-miller ted-miller added bug Something isn't working yrc1000 ros2:humble rv:0.1.3 Reported in MotoROS2 0.1.3 labels Sep 9, 2024
@ted-miller
Copy link
Collaborator Author

And assuming that we can reproduce, add some debug messaging to indicate what's wrong.

@gavanderhoorn
Copy link
Collaborator

Isn't that all part of GP_getFeedbackSpeedMRegisterAddresses(..) (called here)?

@ted-miller
Copy link
Collaborator Author

Yes.
If S1CxG0335 is not set, then GP_getFeedbackSpeedMRegisterAddresses will attempt to directly modify the SC.PRM file in the controller. After that modification, it raises an alarm and sits in an infinite (blocking) loop to force the user to reboot.

But...
a) The user didn't see the alarm. Don't know why...
b) Since the FSU CRC-check was enabled, the modification to SC.PRM was never accepted. So MR2 gets stuck in the bootup routine.

If the pendant isn't properly displaying the alarm, then I don't know that we'll be able to fix that directly. (Will need to report to Japan.) But we can ensure that the debug message gets reported in a way that is visible to the user.

Alternatively (or additionally), we can modify the setup instructions to have users manually enable the speed-feedback parameters.

@gavanderhoorn
Copy link
Collaborator

What about extending GP_getFeedbackSpeedMRegisterAddresses(..) to not

attempt to directly modify the SC.PRM file in the controller. After that modification, it raises an alarm and sits in an infinite (blocking) loop to force the user to reboot.

but instead just return an error? That can then be forwarded / reported to the user in the normal way.

Since the FSU CRC-check was enabled, the modification to SC.PRM was never accepted. So MR2 gets stuck in the bootup routine.

is there a way to detect the FSU is enabled? If there is, we can report that as well.

Alternatively (or additionally), we can modify the setup instructions to have users manually enable the speed-feedback parameters.

I'd really only do that as a last resort. There's quite a few parameters, especially when you have multiple axes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ros2:humble rv:0.1.3 Reported in MotoROS2 0.1.3 yrc1000
Projects
None yet
Development

No branches or pull requests

2 participants