-
Notifications
You must be signed in to change notification settings - Fork 13
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
Frontend Build Instructions #110
Conversation
cw00dw0rd
commented
Apr 29, 2020
- Updated the maintaner.md as it already contained build information. Could be placed somewhere else though.
- May be worthwhile to move any discussion of frontend interaction to the frontend folder. Could offer maintainer information for frontend contributors and have the build steps there instead.
documentation/build/maintainer.md
Outdated
@@ -79,4 +86,27 @@ Edit the the `makefile` to make the following changes: | |||
3. Run `make docker_APSI_build` | |||
|
|||
|
|||
### Building only the ArangoML WebUI |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe match the TOC entry
Building a frontend-only Docker image for the ArangoML WebUI
documentation/build/maintainer.md
Outdated
|
||
2. Run `docker-compose build Dockerfile` | ||
|
||
3. Run `docker run -p 3000:3000 -p 6529:8529 -it arangopipe_frontend_app` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do we need -p 6529:8529?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same with it?
documentation/build/maintainer.md
Outdated
|
||
6. `exit` | ||
|
||
7. Run `docker restart` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
IMo this is not nice, can we just restart the node app
documentation/build/maintainer.md
Outdated
|
||
3. Run `docker ps` to obtain container Name/ID | ||
|
||
4. Run `docker exec -it {Container Name/ID} sh` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we simplify this by setting the env variable via -e on docker run
documentation/build/maintainer.md
Outdated
The following instructions show you how to build the different supported packages. | ||
These instructions include build instructions for: | ||
* Building Arangopipe package | ||
* Building Arangopipe Docker Images with Tensorflow & PyTorch |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/Tensorflow/TensorFlow
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/Image/image