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

Improve documentation in the README #77

Closed
marco-c opened this issue Feb 20, 2018 · 8 comments
Closed

Improve documentation in the README #77

marco-c opened this issue Feb 20, 2018 · 8 comments
Assignees

Comments

@marco-c
Copy link
Owner

marco-c commented Feb 20, 2018

The documentation is pretty scarce, it needs to be improved.

@Shashi456
Copy link
Contributor

@marco-c What exactly are we looking for in the documentation? Should i start with #2 and elaborate on the problem statement first for a start?

@marco-c
Copy link
Owner Author

marco-c commented Feb 20, 2018

I'm working on this myself. If you want, you can improve on the documentation I'm writing.

@Shashi456
Copy link
Contributor

@marco-c Alright

@marco-c
Copy link
Owner Author

marco-c commented Feb 20, 2018

This is now fixed, feel free to suggest additions if you want.

@Shashi456
Copy link
Contributor

@marco-c I had two suggestions for the documentation, one if we can document how to run the project locally? and the other if we could add a few lines at the top which talk about web compatibility briefly and then redirecting to some other page for some extra info.

@marco-c
Copy link
Owner Author

marco-c commented Feb 20, 2018

@marco-c I had two suggestions for the documentation, one if we can document how to run the project locally?

What would you write?

and the other if we could add a few lines at the top which talk about web compatibility briefly and then redirecting to some other page for some extra info.

There's a link now that brings you to a description of Web Compatibility.

@Shashi456
Copy link
Contributor

@marco-c
Something after get dependencies that the train / pretrain.py can be run directly to check how the current network fairs?

and also updating the structure of the project to explain about the rest of the files which have come up recently ? or is that not required?

@marco-c
Copy link
Owner Author

marco-c commented Feb 20, 2018

Something after get dependencies that the train / pretrain.py can be run directly to check how the current network fairs?

I think this should be clear from the description of the structure of the project, but we can add a short sentence.

and also updating the structure of the project to explain about the rest of the files which have come up recently ? or is that not required?

It's not required, but not forbidden either 😄

Feel free to submit PRs for any addition you think worth adding.

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

No branches or pull requests

2 participants