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

Mecademic hardware UI Integration #903

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

TheBigSasha
Copy link
Contributor

A rudimentary version of mecademic support in the hardware UI for Flojoy.
Screenshot 2023-10-02 at 10 21 39 AM

Known issues

  • Mecademic doesn't provide API to scan for arms on the network, and Flojoy hardware integration doesn't allow for hardware that can't be indexed. Because of this, this demo hardcodes the default IP address for mecademic in lieu of a custom connection UI
  • Mecademic's connection handle should in the same Python server as the nodes, so it can't be managed fully externally on the hardware connections server. Because of this, the implementation is just passing around possible IP addresses to connect to

Possible ways forward

  • Enhance hardware connections API to support custom connections (IE a + button). This would allow a user to type in an IP and test the connection in the hardware connections UI, and add it to the dropdown list for the medemic node. This increases consistency but adds friction
  • Do the same as above, but remove the connect node in lieu of implicit connection management through expansion of the previous mecademic connection manager
  • Use the current hardware API as much as possible - scan through all IPs, find one that responds to mecademic requests, hope their API never changes, populate the list of mecademic arms using results.

@jingxiangmo jingxiangmo changed the base branch from main to develop October 2, 2023 17:41
@jingxiangmo
Copy link
Contributor

Need to merge into develop instead of main

@itsjoeoui itsjoeoui changed the base branch from develop to main October 25, 2023 22:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants