-
-
Notifications
You must be signed in to change notification settings - Fork 2
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
Help getting this going #1
Comments
Hello 👋,
After some research I found out this error can throw when an internal error occurred with the server (spot), in our case I think it's just an error in the code : spot-sdk-js/bosdyn-client/robot.js Line 300 in cdde8b9
For testing purpose I used I will patch it soon, if you have any other problem let me know! P.S.: Sorry if my English isn't good. |
It's now fixed (I hope), tell me if this fix the problem! |
Thanks!
I’ll give it a try next week.
Regards,
Ken
… On Oct 15, 2022, at 3:29 PM, Théo Pierné ***@***.***> wrote:
It's now fixed (I hope), tell me if this fix the problem!
—
Reply to this email directly, view it on GitHub <#1 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AADS2PHONIQ47QYRJMTL5A3WDMAZTANCNFSM6AAAAAARFKSPXY>.
You are receiving this because you authored the thread.
|
Unfortunately, it's still getting the same error.
|
I made some research and I found out that can be a problem with the SSL encryption caused by a bad using of the SSL certificate in the code, I started working on a fix but I don't know when I'll be done. I'll let you know when I finish it! |
Good morning! I'm getting a different error now. Let me know if there's anything you want me to try on this end. One of the students here is trying to do a similar thing but with C#. He has not got it completely working yet but he's gotten beyond the GPRC authentication steps so maybe he can look at your code and provide some suggestions as far as how to create the authentication channels and provide the certificates properly. I'l ask him on Wednesday.
|
Hey ! I notice I didn't pass the good credentials and options to the stub constructor spot-sdk-js/bosdyn-client/common.js Lines 215 to 221 in 47edcfc
I think it should work now 🤞 spot-sdk-js/bosdyn-client/common.js Lines 215 to 222 in 51c93ef
In case my modification didn't fix it, add this to the environment variables, rerun
export GRPC_HELLO_SECURE=Y
export GRPC_TRACE=subchannel
export GRPC_VERBOSITY=DEBUG
set GRPC_HELLO_SECURE=Y
set GRPC_TRACE=subchannel
set GRPC_VERBOSITY=DEBUG
No problem, I appreciate your help ! |
Hey we have some progress. lots of output repeating the PowerCommandFeedback messages over and over again and then it throws. It must be getting null back from the power_command_feedback
|
Hey, I'm glad my previous fix is working! I pushed a fix for the If you find any other bug, let me know! (Maybe close this issue and create others one if needed) |
Will try it as soon as possible. Spot is out on a trip today :) |
Also don't forget to run an estop when running Maybe use the official one, my version of the estop doesn't work well.
Well have a good trip Spot! |
Hello Theo,
This module looks awesome and we're excited to try this out here at Code & Circuit. I've got it installed but we're running into a grpc error running hello_spot.js.
Do you have any ideas what this means?
Thanks!
-Ken
The text was updated successfully, but these errors were encountered: