Deep-Forest icon indicating copy to clipboard operation
Deep-Forest copied to clipboard

Better Documentation

Open xuyxu opened this issue 5 years ago • 4 comments

Thanks to the contributors, many new features have been developed. As a result, the current version of documentation could be ambiguous, and requires more explanation or demonstration.

This issue collects suggestions on the documentation. Any one is welcomed to improve the readability of the documentation. For contributors unfamiliar with our workflow on building the documentation, please refer to the instructions below.

Build the documentation locally

  • Clone the repo and install package dependencies:
git clone https://github.com/LAMDA-NJU/Deep-Forest.git
cd Deep-Forest/docs
pip install -r requirements.txt
  • Make modifications to the corresponding .rst file. (Wiki of rst)
  • Build the documentation:
make html

The generated html files are available in the directory _build/html/, and the homepage is index.html.

Build the documentation via pull request

Readthedocs has been integrated into our CI, and you can also view the documentation after creating your PR, available in the last row of GitHub Checks on your PR page.

All contributors

Full list available at Contributors.

  • @dwaipayan05: #31

xuyxu avatar Feb 12 '21 05:02 xuyxu

I've added a Pull Request #31 , can you link the PR to this issue and review the changes.

dwaipayan05 avatar Feb 12 '21 06:02 dwaipayan05

I've added a Pull Request #31 , can you link the PR to this issue and review the changes.

Done.

xuyxu avatar Feb 12 '21 06:02 xuyxu

I think it would be much better to have a page with following things:

  • a tutorial on DF21 model
  • roles of different parameters in DF21 model

Alex-Medium avatar Feb 16 '21 08:02 Alex-Medium

Sorry for my late response @Alex-Medium, I am not aware of your comment.

That would be great, we are going to set up another page under the section Advanced Topics that illustrates the model architecture and the meaning of different parameters.

xuyxu avatar Mar 12 '21 11:03 xuyxu