Skip to content
This repository was archived by the owner on Oct 24, 2023. It is now read-only.

Robotics Nodes #286

Merged
merged 40 commits into from
Sep 28, 2023
Merged

Robotics Nodes #286

merged 40 commits into from
Sep 28, 2023

Conversation

@jingxiangmo
Copy link
Contributor

@jingxiangmo jingxiangmo commented Sep 11, 2023 •
edited
Loading

Description

  • I've included a concise description of what each node does

Styleguide

  • My node adheres to the styleguide for Flojoy nodes

Docs

  • I've submitted a PR for a documentation page for the new node(s) that contains usage examples (see docs.flojoy.io)

Testing

  • This PR includes a unit test (example here and/or ideally a screenshot of the node's output on an example app.

@jingxiangmo jingxiangmo changed the title [WIP] Robotics [WIP] Robotics Nodes Sep 11, 2023
henri123lemoine and others added 27 commits September 11, 2023 04:52
@henri123lemoine
added conn/activate/home/move nodes
5f15eec
@henri123lemoine
added move lin and move pose nodes to mecademic
3ded191
@henri123lemoine
minor refactor
b7a91ca
@henri123lemoine
Moved mecademic nodes to instruments
34ebe00
@henri123lemoine
moved the move nodes
aab2e4d
@henri123lemoine
Added local is-connected checks
789c062
@henri123lemoine
added 2 nodes and updated a few
bd7c439
@henri123lemoine
bug fix
69c631e
@henri123lemoine
This change fixes all CONNECT node errors in manifest generation
e3514ba
@henri123lemoine
Fixed meca nodes return types
5bded07
@henri123lemoine
removed useless imports
784f3a0
@henri123lemoine
move nodes fix
efb38e3
@henri123lemoine
changed move nodes init
cf54202
@TheBigSasha
Add I/O to doc comments for mecademic nodes that didn't previously in…
a1ffdda 
…clude them, and improve doc details
@TheBigSasha
Add I/O to doc comments for mecademic nodes that didn't previously in…
bbc2c74 
…clude them, and improve doc details
@TheBigSasha
Merge remote-tracking branch 'origin/robotics' into robotics
de35066
@TheBigSasha
Merge remote-tracking branch 'origin/robotics' into robotics
de66999
@TheBigSasha
Merge remote-tracking branch 'origin/robotics' into robotics
90ecebc
@henri123lemoine
update set nodes comments
cf1db79
@henri123lemoine
Switched to more precise types
4f7b5f1
@henri123lemoine
set nodes input bugfix
96ea627
@henri123lemoine
move nodes name changes
b73958f
@henri123lemoine
nice cosmetic changes + switch to using MecademicConnHandle
b106162
@henri123lemoine
removed double conn check
240fdb2
@henri123lemoine
undid erroneous changes
4436f67
@henri123lemoine
reset SET_JOINT_VEL conn_handle type
8e4ac57
@TheBigSasha
Add global state based connection handling to basic nodes
2dfd98f
@jingxiangmo jingxiangmo marked this pull request as ready for review September 26, 2023 22:32
@jingxiangmo jingxiangmo changed the title [WIP] Robotics Nodes Robotics Nodes Sep 27, 2023
@jingxiangmo jingxiangmo changed the base branch from main to develop September 27, 2023 18:10
@jingxiangmo jingxiangmo changed the base branch from develop to robotics-nodes September 28, 2023 16:35
@jingxiangmo jingxiangmo merged commit 2e1b7e6 into flojoy-ai:robotics-nodes Sep 28, 2023
@jingxiangmo jingxiangmo deleted the robotics branch September 28, 2023 16:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@jingxiangmo @henri123lemoine @TheBigSasha @dstrande