Skip to content

Latest commit

 

History

History
40 lines (24 loc) · 3.9 KB

CONTRIBUTING.md

File metadata and controls

40 lines (24 loc) · 3.9 KB
Raw

Contributing to this project

👍🎉 First off, thanks for taking the time to contribute! 🎉👍

Opening an issue

I would love to see suggestions. Please:

  • Pay attention to your writing (grammar, style, syntax).
  • Take a look at existing issues in case you are opening a new issue: your problem might already be discussed or solved.
  • If you have a question rather than an issue, and if this question is unrelated to the project or will not be helpful to the community around this project, please contact me in private on LinkedIn or Facebook instead.

Submitting a pull request for improving the code

I would love to see improvements to this code base, as long as things stays clean. By clean, I mean that you:

  • Pay attention to your writing (grammar, style, syntax).
  • Write clean code (e.g.: no magic constants, clean names, no useless comments, clean functions... and on).
  • Seek to apply the boyscout rule.
  • Agree to the conditions of the Apache License 2.0.
  • Respect other people's license(s) in case you use code or contribution is derived or taken from elsewhere on internet.
  • If you are submitting answers to the questions, opening a branch might be a good idea: if so, be sure to give a good name to the branch. Since this project might evolve, don't hesitate to leave comments or some information about the versions of the libraries you are using (e.g.: in the commit message).

That's all. Thanks!

Working with the stock analysis.ipynb file

The instructions on how to open that file are in the README.md and in the stock analysis.ipynb file itself, too. See: https://github.com/romwhyte/jsestockanalysis#how-to-use-this-ipynb-python-notebook-

If your changes to the project are about the structure of the neural network and that it would significantly change the results, please regenerate the charts in the introduction and below, and also ensure to edit the surrounding text. If this is too long or that you don't have the required hardware to do that, you can still open the pull request to let me deal with that later, or maybe someone else will do it. In that case, please specify that the charts need to be regenerated. Those charts are located in the images/ folder, which is different than the stock analysis_files/ folder.

Be sure that the images in the notebook still render, and that you add all the necessary modified files before commiting.

Exporting the stock analysis.ipynb file to README.md and stock analysis.py

The conversion of the stock analysis.ipynb to the README.md is done automatically by running the last cell of the notebook. However, ensure that the generated README file contains the good surrounding metatada. For example, files in the [stock analysis_files/](https://github.com/romwhyte/stock analysis/tree/master/stock analysis) folder might have changed, this folder is used by the generated README to store some of the images. Therefore, I recommend to delete this folder before generating the README.md file anew. Don't forget to add those new generated files to your commit.

The conversion of the stock analysis.ipynb to the README.md is done almost manually. Inside the ipython IDE when in the browser running the notebook, you can use the File > Export menu item (or a sub-menu item with a similar name, under File) to then export the the .py python file format. Once exported, please review the exported file and comment out (or delete) the lines with calls to get_ipython(), because those commands will not run outsite of the ipython notebook environment. Please ensure that this .py file still runs properly. You can then run a git diff *.py in the console to verify your changes before adding and commiting files