Add Dockerfile for containerizing#242
Add Dockerfile for containerizing#242georgemjohnson11 wants to merge 1 commit intoFYNCH-BIO:masterfrom
Conversation
Update README.md to include installation, building, and execution instructions Explain briefly x11 driver requirements
| Windows, can install [Cygwin](https://x.cygwin.com/) with additional configuraiton [here](https://www.kombitz.com/2020/01/31/how-to-configure-cygwin-x-for-remote-connection/) | ||
| A restart may be needed after installation, but a test container can be ran via: | ||
| ```bash | ||
| docker run -p 1212:1212 -v /tmp/.X11-unix:/tmp/.X11-unix -e DISPLAY=$DISPLAY:0 nonasoftware/evolver-electron:latest |
There was a problem hiding this comment.
This can be changed, but just wanted a registry to store the image. I also have some Github Actions scripts I can add in to auto build and upload on PRs, manually, and/or on commit to master
| COPY . . | ||
| RUN yarn --network-timeout 100000 | ||
|
|
||
| EXPOSE 1212 |
There was a problem hiding this comment.
Might need to expose more ports to allow communication like 8081 or the environmental variable ${PORT} could be handled with the ARG/ENV Dockerfile annotations
There was a problem hiding this comment.
I think it should only need a single exposed port to talk to the server on the RPi. The DPU/calibrations scripts that get kicked off via the GUI communicate with the RPi through the same port (<evolver_ip>:8081 by default). The GUI also needs to be able to access the file system of the machine its running on and any mounted drives.
Locations of python scripts being run:
https://github.com/FYNCH-BIO/evolver-electron/blob/master/app/main.dev.js#L85
https://github.com/FYNCH-BIO/evolver-electron/blob/master/app/main.dev.js#L118
And the relevant DPU scripts which are run:
https://github.com/FYNCH-BIO/dpu/blob/master/experiment/template/eVOLVER.py
https://github.com/FYNCH-BIO/dpu/blob/master/calibration/calibrate.py
Update README.md to include installation, building, and execution instructions
Explain briefly x11 driver requirements
What? Why?
Add the ability to run a containerized application for portability and reusability of application
Changes proposed in this pull request:
Checks
Any screenshots or GIFs?
Issue Number: #217