-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[Doc] main doc page update and contribute guide (dmlc#336)
* reorg the doc mainpage * contribute guide
- Loading branch information
1 parent
24bbdb7
commit 6f9ae8d
Showing
16 changed files
with
309 additions
and
61 deletions.
There are no files selected for viewing
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,15 @@ | ||
## Contributing to DGL | ||
|
||
Contribution is always welcomed. A good starting place is the roadmap issue, where | ||
you can find our current milestones. All contributions must go through pull requests | ||
and be reviewed by the committors. See our [contribution guide](https://docs.dgl.ai/contribute.html) for more details. | ||
|
||
Once your contribution is accepted and merged, congratulation, you are now an contributor to the DGL project. | ||
We will put your name in the list below and also on our [website](https://www.dgl.ai/ack). | ||
|
||
Contributors | ||
------------ | ||
[Yizhi Liu](https://github.com/yzhliu) | ||
[Yifei Ma](https://github.com/yifeim) | ||
Hao Jin | ||
[Sheng Zha](https://github.com/szha) |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,136 @@ | ||
Contribute to DGL | ||
================= | ||
|
||
Any contribution to DGL is welcome. This guide covers everything | ||
about how to contribute to DGL. | ||
|
||
General development process | ||
--------------------------- | ||
|
||
A non-inclusive list of types of contribution is as follows: | ||
|
||
* New features and enhancements (`example <https://github.com/dmlc/dgl/pull/331>`__). | ||
* Bugfix (`example <https://github.com/dmlc/dgl/pull/247>`__). | ||
* Document improvement (`example <https://github.com/dmlc/dgl/pull/263>`__). | ||
* New models and examples (`example <https://github.com/dmlc/dgl/pull/279>`__). | ||
|
||
For features and bugfix, we recommend first raise an `issue <https://github.com/dmlc/dgl/issues>`__ | ||
using the corresponding issue template, so that the change could be fully discussed with | ||
the community before implementation. For document improvement and new models, we suggest | ||
post a thread in our `discussion forum <https://discuss.dgl.ai>`__. | ||
|
||
Before development, please first read the following sections about coding styles and testing. | ||
All the changes need to be reviewed in the form of `pull request <https://github.com/dmlc/dgl/pulls>`__. | ||
Our `committors <https://github.com/orgs/dmlc/teams/dgl-team/members>`__ | ||
(who have write permission on the repository) will review the codes and suggest the necessary | ||
changes. The PR could be merged once the reviewers approve the changes. | ||
|
||
Git setup (for developers) | ||
-------------------------- | ||
|
||
First, fork the DGL github repository. Suppose the forked repo is ``https://github.com/username/dgl``. | ||
|
||
Clone your forked repository locally: | ||
|
||
.. code-block:: bash | ||
git clone --recursive https://github.com/username/dgl.git | ||
Setup the upstream to the DGL official repository: | ||
|
||
.. code-block:: bash | ||
git remote add upstream https://github.com/dmlc/dgl.git | ||
You could verify the remote setting by typing ``git remove -v``: | ||
|
||
.. code-block:: bash | ||
origin https://github.com/username/dgl.git (fetch) | ||
origin https://github.com/username/dgl.git (push) | ||
upstream https://github.com/dmlc/dgl.git (fetch) | ||
upstream https://github.com/dmlc/dgl.git (push) | ||
During developing, we suggest work on another branch than the master. | ||
|
||
.. code-block:: bash | ||
git branch working-branch | ||
git checkout working-branch | ||
Once the changes are done, `create a pull request <https://help.github.com/articles/creating-a-pull-request/>`__ | ||
so we could review your codes. | ||
|
||
Once the pull request is merged, update your forked repository and delete your working branch: | ||
|
||
.. code-block:: bash | ||
git checkout master | ||
git pull upstream master | ||
git push origin master # update your forked repo | ||
git branch -D working-branch # the local branch could be deleted | ||
Coding styles | ||
------------- | ||
|
||
For python codes, we generally follow the `PEP8 style guide <https://www.python.org/dev/peps/pep-0008/>`__. | ||
The python comments follow `NumPy style python docstrings <https://sphinxcontrib-napoleon.readthedocs.io/en/latest/example_numpy.html>`__. | ||
|
||
For C++ codes, we generally follow the `Google C++ style guide <https://google.github.io/styleguide/cppguide.html>`__. | ||
The C++ comments should be `Doxygen compatible <http://www.doxygen.nl/manual/docblocks.html#cppblock>`__. | ||
|
||
Coding styles check is mandatory for every pull requests. To ease the development, please check it | ||
locally first (require cpplint and pylint to be installed first): | ||
|
||
.. code-block:: bash | ||
bash tests/scripts/task_lint.sh | ||
The python code style configure file is ``tests/scripts/pylintrc``. We tweak it a little bit from | ||
the standard. For example, following variable names are accepted: | ||
|
||
* ``i,j,k``: for loop variables | ||
* ``u,v``: for representing nodes | ||
* ``e``: for representing edges | ||
* ``g``: for representing graph | ||
* ``fn``: for representing functions | ||
* ``n,m``: for representing sizes | ||
* ``w,x,y``: for representing weight, input, output tensors | ||
* ``_``: for unused variables | ||
|
||
Building and Testing | ||
-------------------- | ||
|
||
To build DGL locally, follow the steps described in :ref:`Install from source <install-from-source>`. | ||
However, to ease the development, we suggest NOT install DGL but directly working in the source tree. | ||
To achieve this, export following environment variables: | ||
|
||
.. code-block:: bash | ||
export DGL_HOME=/path/to/your/dgl/clone | ||
export DGL_LIBRARY_PATH=$DGL_HOME/build | ||
export PYTHONPATH=$PYTHONPATH:$DGL_HOME/python | ||
You could test the build by running the following command and see the path of your local clone. | ||
|
||
.. code-block:: bash | ||
python -c 'import dgl; print(dgl.__path__)' | ||
TBD by Quan about how to run and write unittests. | ||
|
||
Building documents | ||
------------------ | ||
|
||
If the change is about document improvement, we suggest build the document and render it locally | ||
before pull request. See instructions `here <https://github.com/dmlc/dgl/tree/master/docs>`__. | ||
|
||
Data hosting | ||
------------ | ||
|
||
If the change is about new models or applications, it is very common to have some data files. Data | ||
files are not allowed to be uploaded to our repository. Instead, they should be hosted on the | ||
cloud storage service (e.g. dropbox, Amazon S3) and downloaded on-the-fly. See our :ref:`dataset APIs <apidata>` | ||
for more details. All the dataset of current DGL models are hosted on Amazon S3. If you want your | ||
dataset to be hosted as well, please post in our `discussion forum <https://discuss.dgl.ai>`__. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,6 @@ | ||
Resources | ||
========= | ||
* If you are new to deep learning, `Dive into Deep Learning <http://diveintodeeplearning.org>`__ | ||
is a nice book to start with. | ||
* `Pytorch tutorials <https://pytorch.org/tutorials/>`__ | ||
* Thomas Kipf's `blog on Graph Convolutional Networks <https://tkipf.github.io/graph-convolutional-networks/>`__ |
Oops, something went wrong.