onshape-to-robot icon indicating copy to clipboard operation
onshape-to-robot copied to clipboard

Pulling from Branches

Open ThomasPiechulek opened this issue 4 years ago • 9 comments

Hello, i can not find a possibility to onshape-to-robot on a specified branch. The script seems to just take the last one active. Can you add the Workspace Id in the Config.json to solve this ?

ThomasPiechulek avatar Feb 25 '21 11:02 ThomasPiechulek

Hello, Can you use versionId parameter for that ? https://onshape-to-robot.readthedocs.io/en/latest/config.html#versionid

Gregwar avatar Feb 25 '21 13:02 Gregwar

Currently, versionId can be used to specify a version (/v/), I can also add a workspaceId to be used /w/ if you need it

Gregwar avatar Feb 25 '21 13:02 Gregwar

Hello, yes that would be great. Beacause we try to do everything automatically, and with the version id, we have to manually create versions every time... Thank you so much.


Von: Grégoire Passault [email protected] Gesendet: Donnerstag, 25. Februar 2021 13:23 An: Rhoban/onshape-to-robot [email protected] Cc: Thomas Piechulek [email protected]; Author [email protected] Betreff: Re: [Rhoban/onshape-to-robot] Pulling from Branches (#47)

Currently, versionId can be used to specify a version (/v/), I can also add a workspaceId to be used /w/ if you need it

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/Rhoban/onshape-to-robot/issues/47#issuecomment-785891565, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AS7TIQ4QW4VDWY4OCBY5DZTTAZFN5ANCNFSM4YGKGPYQ.

ThomasPiechulek avatar Feb 26 '21 12:02 ThomasPiechulek

Just added workspaceId configuration entry, can you confirm it's working for you ?

Gregwar avatar Feb 26 '21 13:02 Gregwar

works fine, thank you.


Von: Grégoire Passault [email protected] Gesendet: Freitag, 26. Februar 2021 13:24 An: Rhoban/onshape-to-robot [email protected] Cc: Thomas Piechulek [email protected]; Author [email protected] Betreff: Re: [Rhoban/onshape-to-robot] Pulling from Branches (#47)

Just added workspaceId configuration entry, can you confirm it's working for you ?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/Rhoban/onshape-to-robot/issues/47#issuecomment-786645809, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AS7TIQ5BKKTDBJNZGKMAQJ3TA6OI7ANCNFSM4YGKGPYQ.

ThomasPiechulek avatar Mar 01 '21 11:03 ThomasPiechulek

In the code we are searching for the ID, but it wants to print the Keyword Workspace, so some kollegues have this error occuring. Strangely this Eroor does not appear for me so sorry for the early confirm.

Traceback (most recent call last): File ".../bin/onshape-to-robot", line 3, in from onshape_to_robot import onshape_to_robot File "...lib/python3.8/site-packages/onshape_to_robot/onshape_to_robot.py", line 14, in from .load_robot import
File ".../lib/python3.8/site-packages/onshape_to_robot/load_robot.py", line 19, in config['workspace']+' ...' + Style.RESET_ALL) KeyError: 'workspace'

ThomasPiechulek avatar Mar 02 '21 09:03 ThomasPiechulek

The problem seems to be in the load_robot.py file at ll. 18 ff., where the config is accessed with a faulty key and later on the workspaceId variable is required. The code snippet below contains my edits with which the code worked. workspaceId=config['workspaceId'] print("\n" + Style.BRIGHT + '* Using configuration workspace ID ' + config['workspaceId']+' ...' + Style.RESET_ALL) By the way, great work you're doing here!

isetre avatar Mar 02 '21 10:03 isetre

You are right, sorry Should be fixed in v0.3.9

Gregwar avatar Mar 02 '21 11:03 Gregwar

works fine now, thanks

isetre avatar Mar 02 '21 14:03 isetre