Skip to content

Commit d057602

Browse files
matteocaoGuillaume Tauzin
authored andcommitted
Documents rst (giotto-ai#17) (giotto-ai#20)
* Update azure-pipelines.yml * Update azure-pipelines.yml * Create build_manylinux2010.sh * Update azure-pipelines.yml * Update build_manylinux2010.sh * Update build_manylinux2010.sh * Update azure-pipelines.yml * Update azure-pipelines.yml * Update azure-pipelines.yml * Update azure-pipelines.yml * Update azure-pipelines.yml * Update azure-pipelines.yml * Update azure-pipelines.yml * Update azure-pipelines.yml * Update azure-pipelines.yml * Update azure-pipelines.yml * Update azure-pipelines.yml * Update azure-pipelines.yml * Update azure-pipelines.yml * Update azure-pipelines.yml * Update azure-pipelines.yml * Update azure-pipelines.yml * Update azure-pipelines.yml * Update azure-pipelines.yml * Update azure-pipelines.yml * Update azure-pipelines.yml * Update azure-pipelines.yml * Update azure-pipelines.yml * Update azure-pipelines.yml * Update azure-pipelines.yml * Update build_manylinux2010.sh * Update azure-pipelines.yml * Create docker_scripts.sh * Update build_manylinux2010.sh * Create build_manylinux.sh * Create docker_scripts.sh * Rename build_manylinux.sh to build_manylinux2010.sh * Update build_manylinux2010.sh * Update docker_scripts.sh * Delete build_manylinux2010.sh * Delete docker_scripts.sh * Update azure-pipelines.yml * Update azure-pipelines.yml * Update azure-pipelines.yml * Update build_manylinux2010.sh * Update build_manylinux2010.sh * Update build_manylinux2010.sh * Update docker_scripts.sh * Update azure-pipelines.yml * Update docker_scripts.sh * Update docker_scripts.sh * Update docker_scripts.sh * Update azure-pipelines.yml * Update build_manylinux2010.sh * Update docker_scripts.sh * Update docker_scripts.sh * Update docker_scripts.sh * Update docker_scripts.sh * Update docker_scripts.sh * Update docker_scripts.sh * Update docker_scripts.sh * Update docker_scripts.sh * Update docker_scripts.sh * Update docker_scripts.sh * Update docker_scripts.sh * Update docker_scripts.sh * Update docker_scripts.sh * Update docker_scripts.sh * Update docker_scripts.sh * Update docker_scripts.sh * Update azure-pipelines.yml * Update docker_scripts.sh * Update build_manylinux2010.sh * Update docker_scripts.sh * Update docker_scripts.sh * Update docker_scripts.sh * Update docker_scripts.sh * Update docker_scripts.sh * Update docker_scripts.sh * Update docker_scripts.sh * Update docker_scripts.sh * Update docker_scripts.sh * Update docker_scripts.sh * Update docker_scripts.sh * Update docker_scripts.sh * Update docker_scripts.sh * Update CMakeLists.txt * Update docker_scripts.sh * Update docker_scripts.sh * Update docker_scripts.sh * Update docker_scripts.sh * Update docker_scripts.sh * Update docker_scripts.sh * Update docker_scripts.sh * Update docker_scripts.sh * Update docker_scripts.sh * Update docker_scripts.sh * Update docker_scripts.sh * Update docker_scripts.sh * Update docker_scripts.sh * Update docker_scripts.sh * Update docker_scripts.sh * Update docker_scripts.sh * Fixed manylinux wheels script (#2) * Fixed boost path for manylinux wheels (#2) * Update CMakeLists.txt * Update build_manylinux2010.sh * Update docker_scripts.sh * Update azure-pipelines.yml * Update docker_scripts.sh * Update azure-pipelines.yml * Update README.rst * Update README.rst * Update README.rst * Update README.rst * Removed pybind11 from manylinux wheels script (#2) * Update README.rst * solve a windows compatibility issue of .rst file * Delete Lorenz_Attractor.ipynb * update the Lorenz_attractor update the example file with dataset from openML * Delete Classifying_shapes.ipynb * update classifying_shapes working version of this example: done! * update binder environment update binder environment with OpenML. * Update azure-pipelines.yml * Update README.rst * Update README.rst * Update README.rst * Delete AUTHORS * Delete GOVERNANCE.rst * Delete DEED_OF_CONTRIBUTIONS.rst * Delete CODE_OF_CONDUCT.rst * Update LICENSE * Add files via upload
1 parent b7609bf commit d057602

File tree

8 files changed

+203
-76
lines changed

8 files changed

+203
-76
lines changed

AUTHORS

Lines changed: 0 additions & 10 deletions
This file was deleted.

CODE_AUTHORS.rst

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
# The following is the list of the code authors of the giotto-learn python
2+
# package. Where component authors are known, add them here.
3+
4+
L2F SA
5+
EPFL - Ecole Polytechnique Fédérale de Lausanne
6+
REDS Institute of the Haut Ecole d'Ingénierie et Gestion du canton Vaud

CODE_OF_CONDUCT.rst

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -45,11 +45,11 @@ Examples of representing a project or community include using an official projec
4545
Enforcement
4646
-----------
4747

48-
Instances of abusive, harassing, or otherwise unacceptable behavior may be reported by contacting the project team at <[email protected]>. All complaints will be reviewed and investigated and will result in a response that is deemed necessary and appropriate to the circumstances, Sanctions may include written warnings, expulsions from the project, project sponsored spaces, or project forums, or any other sanction which is deemed appropriate. [The project team] is obligated to maintain confidentiality with regard to the reporter of an incident. If the act is ongoing (such as someone engaging in harassment) or involves a threat to anyone's safety (e.g. threats of violence), the the project team may issue sanctions without notice. Further details of specific enforcement policies may be posted separately.
48+
Instances of abusive, harassing, or otherwise unacceptable behavior may be reported by contacting the project team at <[email protected]>. All complaints will be reviewed and investigated and will result in a response that is deemed necessary and appropriate to the circumstances, Sanctions may include written warnings, expulsions from the project, project sponsored spaces, or project forums, or any other sanction which is deemed appropriate. [The project team] is obligated to maintain confidentiality with regard to the reporter of an incident. If the act is ongoing (such as someone engaging in harassment) or involves a threat to anyone's safety (e.g. threats of violence), the the project team may issue sanctions without notice. Further details of specific enforcement policies may be posted separately.
4949

50-
Project maintainers who do not follow or enforce the Code of Conduct in good faith may face temporary or permanent repercussions as determined by other members of the project’s leadership.
50+
Project maintainers who do not follow or enforce the Code of Conduct in good faith may face temporary or permanent repercussions as determined by the project leader.
5151

5252
Attribution
5353
-----------
5454

55-
This Code of Conduct is adapted from the Contributor Covenant, version 1.4, available at <https://www.contributor-covenant.org/version/1/4/code-of-conduct.html>, and includes some aspects of the Giotto Code of Conduct, available at <https://github.com/giotto-learn/giotto-learn/blob/master/CODE_OF_CONDUCT.rst>
55+
This Code of Conduct is adapted from the Contributor Covenant, version 1.4, available at <https://www.contributor-covenant.org/version/1/4/code-of-conduct.html>, and includes some aspects of the TensorFlow Code of Conduct, available at <https://github.com/tensorflow/tensorflow/blob/master/CODE_OF_CONDUCT.md>

CONTRIBUTING.rst

Lines changed: 103 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,103 @@
1+
Contributing guidelines
2+
=======================
3+
4+
Pull Request Checklist
5+
----------------------
6+
7+
Before sending your pull requests, make sure you followed this list.
8+
- Read [contributing guidelines](GOVERNANCE.rst).
9+
- Read [Code of Conduct](CODE_OF_CONDUCT.rst).
10+
- Ensure you have signed the [Contributor License Agreement (CLA)](DEED_OF_CONTRIBUTIONS.rst).
11+
- Check if the changes are consistent with the guidelines.
12+
- Changes are consistent with the Coding Style.
13+
- Run Unit Tests.
14+
15+
How to become a contributor and submit your own code
16+
----------------------------------------------------
17+
18+
Contributor License Agreements
19+
------------------------------
20+
21+
In order to become a contributor of Giotto, the first step is to fill out the
22+
[Contributor License Agreement (CLA)](DEED_OF_CONTRIBUTIONS.rst).
23+
Ican sign the document and send it to <[email protected]>. Once we receive
24+
it, we'll be able to accept your pull requests.
25+
**NOTE**: Only original source code from you and other people that have signed
26+
the CLA can be accepted into the main repository.
27+
28+
Contributing code
29+
-----------------
30+
31+
If you have improvements to Giotto, do not hesitate to send us pull requests!
32+
Please follow the Github [how to](https://help.github.com/articles/using-pull-requests/).
33+
The Giotto Team will review your pull requests. Once the pull requests are approved and pass continuous integration checks, the
34+
Giotto team will work on getting your pull request submitted to our GitHub
35+
repository. Eventually, your pull request will be merged automatically on GitHub.
36+
If you want to contribute, start working through the Giotto codebase,
37+
navigate to the [Github "issues" tab](https://github.com/giotto-learn/giotto-learn/issues)
38+
and start looking through interesting issues. These are issues that we believe
39+
are particularly well suited for outside contributions, often because we
40+
probably won't get to them right now. If you decide to start on an issue, leave
41+
a comment so that other people know that you're working on it. If you want to
42+
help out, but not alone, use the issue comment thread to coordinate.
43+
44+
Contribution guidelines and standards
45+
-------------------------------------
46+
47+
Before sending you pull request for review, make sure your changes are
48+
consistent with the guidelines and follow the coding style below.
49+
50+
General guidelines and philosophy for contribution
51+
--------------------------------------------------
52+
53+
* Include unit tests when you contribute new features, as they help to
54+
a) prove that your code works correctly, and
55+
b) guard against future breaking changes to lower the maintenance cost.
56+
* Bug fixes also generally require unit tests, because the presence of bugs
57+
usually indicates insufficient test coverage.
58+
* Keep API compatibility in mind when you change code in core Giotto.
59+
* Clearly define your exceptions using the utils functions and test the exceptions.
60+
* When you contribute a new feature to Giotto, the maintenance burden is   
61+
(by default) transferred to the Giotto team. This means that the benefit   
62+
of the contribution must be compared against the cost of maintaining the   
63+
feature.
64+
65+
C++ coding style
66+
----------------
67+
68+
Changes to Giotto C/C++ code should conform to [Google C++ Style Guide](https://google.github.io/styleguide/cppguide.html).
69+
Use `clang-tidy` to check your C/C++ changes. To install `clang-tidy` on
70+
ubuntu:16.04, do:
71+
72+
.. code-block:: python
73+
74+
bashapt-get install -y clang-tidy
75+
76+
You can check a C/C++ file by doing:
77+
78+
.. code-block:: python
79+
80+
bashclang-format <my_cc_file> --style=google > /tmp/my_cc_file.ccdiff <my_cc_file> /tmp/my_cc_file.cc
81+
82+
Python coding style
83+
-------------------
84+
85+
Changes to Giotto Python code should conform to PEP8 directives.
86+
Use `flake8` to check your Python changes. To install `flake8` just do
87+
88+
.. code-block:: python
89+
90+
pip install flake8
91+
92+
You can use `flake8` on your python code via the following instructions:
93+
.. code-block:: python
94+
95+
flake8 name_of_your_script.py
96+
97+
Running unit tests
98+
------------------
99+
100+
There are two ways to run Giotto unit tests.
101+
1.  Using tools and libraries installed directly on your system. The election
102+
tool is pytest.
103+
2.  Using [Azure](azure-pipelines.yml) and Giotto's CI scripts.  

DEED_OF_CONTRIBUTIONS.rst renamed to DEED_OF_CONTRIBUTION.rst

Lines changed: 9 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
1-
L2F – Contributor Assignment Agreement
2-
======================================
1+
L2F – Contributor License Agreement
2+
===================================
33

4-
In order to clarify the intellectual property license granted with Contributions from any person or entity, L2F SA, Bâtiment D EPFL Innovation Park, 1015 Lausanne, Switzerland (**L2F**) must have a Contributor Assignment Agreement on file that has been signed by each Contributor, indicating agreement to the terms below. This agreement (the **Agreement**) is for Your protection as a contributor as well as the protection of L2F.
4+
In order to clarify the intellectual property license granted with Contributions from any person or entity, L2F SA, Bâtiment D EPFL Innovation Park, 1015 Lausanne, Switzerland (**L2F**) must have a Contributor License Agreement on file that has been signed by each Contributor, indicating agreement to the terms below. This agreement (the **Agreement**) is for Your protection as a contributor as well as the protection of L2F.
55

66
You accept and agree to the following terms and conditions for Your present and future Contributions submitted to L2F. Except for the rights granted herein to L2F, and (at L2F’s discretion) recipients of software made available by L2F, You reserve all right, title, and interest in and to Your Contributions.
77

@@ -14,13 +14,11 @@ You accept and agree to the following terms and conditions for Your present and
1414

1515
**Your Essential Patent Claims** shall mean all patent claims owned or controlled by You, whether already acquired or hereafter acquired, that would be infringed by some manner of making, using, or selling Your Contribution, but do not include claims that would be infringed only as a consequence of further modification of Your Contribution. For purposes of this definition, "control" includes the right to grant patent sublicenses in a manner consistent with the requirements of this Agreement.
1616

17-
2. Copyrights
17+
2. Copyrights
1818
--------------
1919

20-
(a) At the time Your Contribution is Submitted, You assign to L2F all right, title, and interest worldwide in all rights of authorship and other copyrights covering Your Contribution.
21-
(b) To the extent that any of the rights in Section 2.1 cannot be assigned by You to L2F, You grant to L2F a perpetual, worldwide, non-exclusive, no-charge, royalty-free, transferable, irrevocable license under such non-assigned rights, with rights to sublicense through multiple tiers of sublicensees, to practice such non-assigned rights, including, but not limited to, the right to reproduce, modify, prepare derivative works of, publicly display, publicly perform, propagate, sublicense, and distribute Your Contribution.
22-
(c) To the extent that any of the rights in Section 2.1 can neither be assigned nor licensed by You to L2F, You irrevocably waive and agree never to assert such rights against L2F, any of L2F’s successors in interest, or any of L2F’s licensees, either direct or indirect.
23-
(d) Upon such transfer of rights to L2F, to the maximum extent possible, L2F immediately grants to You a perpetual, worldwide, non-exclusive, no-charge, royalty-free, transferable, irrevocable license under such rights covering Your Contribution, with rights to sublicense through multiple tiers of sublicensees, to reproduce, modify, prepare derivative works of, publicly display, publicly perform, propagate, sublicense, and distribute Your Contribution. The intention of the parties hereto is that this license will be as broad as possible and to provide You with rights as similar as possible to the owner of the rights that You transferred. This license back is limited to Your Contribution and does not provide any further rights to the Work.
20+
Subject to the terms and conditions of this Agreement, You hereby grant to L2F and to recipients of software distributed by L2F (at L2F’s discretion) a perpetual, worldwide, non-exclusive, no-charge, royalty-free, irrevocable copyright license to reproduce, modify, prepare derivative works of, publicly display, publicly perform, propagate, sublicense, and distribute Your Contributions and such derivative works.
21+
To the extent that any of such rights cannot be licensed by You to L2F, You irrevocably waive and agree never to assert such rights against L2F, any of L2F’s successors in interest, or any of L2F’s licensees, either direct or indirect.
2422

2523
3. Patents
2624
-----------
@@ -40,18 +38,18 @@ You agree to notify L2F of any facts or circumstances of which You become aware
4038

4139
You are not expected to provide support for Your Contributions, except to the extent You desire to provide support. You may provide support for free, for a fee, or not at all. Unless required by applicable law or agreed to in writing, You provide Your Contributions on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied, including, without limitation, any warranties or conditions of TITLE, NON- INFRINGEMENT, MERCHANTABILITY, or FITNESS FOR A PARTICULAR PURPOSE.
4240

43-
6. Submissions on behalf of others
41+
6. Submissions on behalf of others
4442
-----------------------------------
4543

4644
Should You wish to submit work that is not Your original creation, You may submit it to L2F separately from any Contribution, identifying the complete details of its source and of any license or other restriction (including, but not limited to, related patents, trademarks, and license agreements) of which You are personally aware, and conspicuously marking the work as "Submitted on behalf of a third-party: [named here]".
4745

4846
7. L2F’s Rights
4947
----------------
5048

51-
(a) No Duty to Use. You acknowledge that L2F is not obligated to use Your Contribution as part of a Work and may decide to include any Contribution it considers appropriate.
49+
(a) No Duty to Use. You acknowledge that L2F is not obligated to use Your Contribution as part of a Work and may decide to include any Contribution it considers appropriate.
5250
(b) Outbound License. If L2F includes Your Contribution in a Work, it may license Your Contribution under any license, including copyleft, permissive, commercial, or proprietary licenses.
5351

54-
8. Miscellaneous
52+
8. Miscellaneous
5553
-----------------
5654

5755
(a) *Severability*. If any provision of this Agreement is held to be invalid or unenforceable for any reason, the parties hereto shall replace it by a substitute provision that achieves to the fullest extent possible the same legal and economic purposes as those of the invalid or unenforceable provision. In any event, the remainder of this Agreement shall remain in full force and effect between the parties,

GOVERNANCE.rst

Lines changed: 14 additions & 44 deletions
Original file line numberDiff line numberDiff line change
@@ -1,50 +1,20 @@
1-
# Contributing guidelines
2-
## Pull Request Checklist
3-
Before sending your pull requests, make sure you followed this list.
4-
- Read [contributing guidelines](GOVERNANCE.md).
5-
- Read [Code of Conduct](CODE_OF_CONDUCT.rst).
6-
- Ensure you have signed the [Contributor License Agreement (CLA)](DEED_OF_CONTRIBUTIONS.rst).
7-
- Check if my changes are consistent with the guidelines.
8-
- Changes are consistent with the Coding Style.
9-
- Run Unit Tests.
10-
## How to become a contributor and submit your own code
11-
### Contributor License Agreements
12-
We'd love to accept your patches! Before we can take them, we have to jump a couple of legal hurdles.
13-
Please fill out either the individual or corporate Contributor License Agreement (CLA).
14-
  * If you are an individual writing original source code and you're sure you own the intellectual property, then you'll need to sign an [individual CLA](DEED_OF_CONTRIBUTIONS.rst).
15-
  * If you work for a company that wants to allow you to contribute your work, then you'll need to sign a corporate CLA (please write at <[email protected]>).
16-
Follow either of the two links above to access the appropriate CLA and instructions for how to sign and return it. Once we receive it, we'll be able to accept your pull requests.
17-
***NOTE***: Only original source code from you and other people that have signed the CLA can be accepted into the main repository.
18-
### Contributing code
19-
If you have improvements to Giotto, send us your pull requests! For thosejust getting started, Github has a[how to](https://help.github.com/articles/using-pull-requests/).
20-
Giotto team members will be assigned to review your pull requests. Once thepull requests are approved and pass continuous integration checks, a Giotto team member will apply `ready to pull` label to your change. This means we areworking on getting your pull request submitted to our internal repository. Afterthe change has been submitted internally, your pull request will be mergedautomatically on GitHub.
21-
If you want to contribute, start working through the Giotto codebase,navigate to the [Github "issues" tab](https://github.com/giotto-learn/giotto-learn/issues) and startlooking through interesting issues. These are issues that we believe are particularly well suited for outsidecontributions, often because we probably won't get to them right now. If youdecide to start on an issue, leave a comment so that other people know thatyou're working on it. If you want to help out, but not alone, use the issuecomment thread to coordinate.
22-
### Contribution guidelines and standards
23-
Before sending your pull request for review, make sure your changes are consistent with the guidelines and follow the Giotto coding style.¯
24-
#### General guidelines and philosophy for contribution
25-
* Include unit tests when you contribute new features, as they help to a)    prove that your code works correctly, and b) guard against future breaking    changes to lower the maintenance cost.*   Bug fixes also generally require unit tests, because the presence of bugs    usually indicates insufficient test coverage.*   Keep API compatibility in mind when you change code in core Giotto. Reviewers of    your pull request will comment on any API compatibility issues.*   When you contribute a new feature to Giotto, the maintenance burden is    (by default) transferred to the Giotto team. This means that the benefit    of the contribution must be compared against the cost of maintaining the    feature.
26-
* Full new features (e.g., a new op implementing a cutting-edge algorithm)    typically will live in    [giotto-learn/addons](https://github.com/giotto-learn/addons) to get some    airtime before a decision is made regarding whether they are to be migrated to the core.
27-
#### C++ coding style
28-
Changes to Giotto C/C++ code should conform to [Google C++ Style Guide](https://google.github.io/styleguide/cppguide.html).
29-
Use `clang-tidy` to check your C/C++ changes. To install `clang-tidy` on ubuntu:16.04, do:
1+
This file describe the governance of the Giotto project.
302

31-
```bashapt-get install -y clang-tidy```
3+
Project owner:
4+
--------------
325

33-
You can check a C/C++ file by doing:
6+
- L2F SA
347

35-
```bashclang-format <my_cc_file> --style=google > /tmp/my_cc_file.ccdiff <my_cc_file> /tmp/my_cc_file.cc```
8+
Authors:
9+
--------
3610

37-
#### Python coding style
38-
Changes to Giotto Python code should conform to [Google Python Style Guide](https://github.com/google/styleguide/blob/gh-pages/pyguide.md)
39-
Use `pylint` to check your Python changes. To install `pylint` and retrieve Giotto's custom style definition:
11+
- Please refer to the [authors](CODE_AUTHORS) file
4012

41-
```bash pip install pylintwget -O /tmp/pylintrc https://raw.githubusercontent.com/giotto-learn/giotto-learn/master/giotto/tools/ci_build/pylintrc```
13+
Giotto Project Team:
14+
--------------------
4215

43-
To check a file with `pylint`:
44-
45-
```bashpylint --rcfile=/tmp/pylintrc myfile.py```
46-
47-
#### Running unit tests
48-
There are two ways to run Giotto unit tests.
49-
1.  Using tools and libraries installed directly on your system. The election tool is pytest.
50-
2.  Using [Azure](azure-pipelines.yml) and TensorFlow's CI scripts.  
16+
- Guillaume Tauzin [email protected] (Maintainer & Main Developer)
17+
- Umberto Lupo [email protected] (Maintainer & Developer)
18+
- Matteo Caorsi [email protected] (Project leader)
19+
- Philippe Nguyen [email protected] (Developer)
20+
- Alessio Ghiraldello [email protected] (Developer)

0 commit comments

Comments
 (0)