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

Virtual Terminal Support #108

Open
aaron97neu opened this issue Oct 27, 2020 · 0 comments
Open

Virtual Terminal Support #108

aaron97neu opened this issue Oct 27, 2020 · 0 comments
Labels
enhancement New feature or request

Comments

@aaron97neu
Copy link
Member

aaron97neu commented Oct 27, 2020

Based on 10/27/20 call with @wang701.

Integrating with the CAN based virtual terminal would be a huge feature. It would allow us to communicate with the operator and allow for much better debugging on machine. If we are going to give back to the farmers who allow us to host our machines by providing them with additional functionality to their machines this will be critical

Based on my call with @wang701 we will need to download the ISOAgLib library and compile that with whatever VT code we write ourselves using cmake. Since this is compiled code, a two stage build is appropriate: build the code on the first container and then move the binary and dynamically linked libraries over to the slimmed runner container. There does not appear to be many dependencies beyond the ISOAgLib stuff

Potential issues:

  • ISOAgLib Is currently going a transition as they are being purchased. Their website and other things are not stable and may cause issues with us programmatically downloading the library as things are moved around or no longer available. Will we want to host our own mirror of the library version that we use?
  • This will require a connection to socketcan. Will this require running in host mode?
  • What base container should we use? This uses cmake extensively, but there is not a dedicated cmake docker base container, we could use gcc for the builder (if that helps?) or just debian-buster
  • Can we test this or programmatically verify that the container is running properly?
@aaron97neu aaron97neu added the enhancement New feature or request label Oct 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant