epvienneau / skin_lesion_classification_app

MIT License
0 stars 0 forks source link

Documentation #57

Closed polortiz4 closed 6 years ago

polortiz4 commented 6 years ago

Not sure if these should be done

epvienneau commented 6 years ago

Yes this should be done

epvienneau commented 6 years ago
screen shot 2017-12-13 at 10 05 29 pm

Cannot figure this one out...

suyashkumar commented 6 years ago

Ah...so this is because tensorflow is installed only inside the Docker container but the docs are being built outside the Docker container. One way to fix this is to install tensorflow globally on the machine where you are building docs pip install tensorflow.

Another solution would be to have the make html run inside the Docker container by exec'ing in...but no worries on that front, installing tensorflow on the machine to build docs should be fine.

suyashkumar commented 6 years ago

you could also try to run the docs process inside the docker container by doing the following, but it's definitely a little more work...

docker ps to find the server container id then docker exec -it <server container id> bash (now you're inside the container where tensorflow is available) assuming you've installed sphinx already (pip install Sphinx) you should be able to just run all the needed commands...

epvienneau commented 6 years ago

Thanks Suyash! Pablo somehow got this working. even pulling down his Pablo/Sphinx branch and following his same steps, it still doesn't work for me, so I'm going to let him handle this one.

polortiz4 commented 6 years ago

@mlp6 @suyashkumar .. From the Sphinx guidelines, we're supposed to gitignore any _build/ directory, but... If so, how can you see the documentation?

mlp6 commented 6 years ago

End user would build it in the format they want with the Makefile!

polortiz4 commented 6 years ago

But that means they would need to have all the dependencies installed, right?

polortiz4 commented 6 years ago

And with Emelina we noticed that’s a little more complicated than activating the environment

polortiz4 commented 6 years ago

@epvienneau

epvienneau commented 6 years ago

Yea I was unable to generate the docs for myself actually :(