XP-227 - Merge v0.3.0 changes into master

.dockerignore

@@ -1,143 +1,23 @@
-# Project
-results
-output
-docs
-
-# Git
-.git
-
-# Pantsbuild
-.pants.d
-.pants.workdir.file_lock
-.pants.workdir.file_lock.lock_message
-
-# Byte-compiled / optimized / DLL files
-__pycache__/
-*.py[cod]
-*$py.class
-
-# C extensions
-*.so
-
-# Distribution / packaging
+**/__pycache__
+*,cover
+*.log
+*.pyc
+*.pyd
+*.pyo
 .Python
-build/
-develop-eggs/
-#dist/
-downloads/
-eggs/
-.eggs/
-lib/
-lib64/
-parts/
-sdist/
-var/
-wheels/
-pip-wheel-metadata/
-share/python-wheels/
-*.egg-info/
-.installed.cfg
-*.egg
-MANIFEST
-
-# PyInstaller
-#  Usually these files are written by a python script from a template
-#  before PyInstaller builds the exe, so as to inject date/other infos into it.
-*.manifest
-*.spec
-
-# Installer logs
-pip-log.txt
-pip-delete-this-directory.txt
-
-# Unit test / coverage reports
-htmlcov/
-.tox/
-.nox/
+.cache
 .coverage
 .coverage.*
-.cache
-nosetests.xml
+.git
+.github
+.tox
+Dockerfile
 coverage.xml
-*.cover
-.hypothesis/
-.pytest_cache/
-
-# Translations
-*.mo
-*.pot
-
-# Django stuff:
-*.log
-local_settings.py
-db.sqlite3
-
-# Flask stuff:
-instance/
-.webassets-cache
-
-# Scrapy stuff:
-.scrapy
-
-# Sphinx documentation
-docs/_build/
-
-# PyBuilder
-target/
-
-# Jupyter Notebook
-.ipynb_checkpoints
-
-# IPython
-profile_default/
-ipython_config.py
-
-# pyenv
-.python-version
-
-# pipenv
-#   According to pypa/pipenv#598, it is recommended to include Pipfile.lock in version control.
-#   However, in case of collaboration, if having platform-specific dependencies or dependencies
-#   having no cross-platform support, pipenv may install dependencies that don’t work, or not
-#   install all needed dependencies.
-#Pipfile.lock
-
-# celery beat schedule file
-celerybeat-schedule
-
-# SageMath parsed files
-*.sage.py
-
-# Environments
-.env
-.venv
-env/
+docs
+env
+nosetests.xml
+pip-delete-this-directory.txt
+pip-log.txt
+scripts
+tests
 venv/
-ENV/
-env.bak/
-venv.bak/
-
-# Spyder project settings
-.spyderproject
-.spyproject
-
-# Rope project settings
-.ropeproject
-
-# mkdocs documentation
-/site
-
-# mypy
-.mypy_cache/
-.dmypy.json
-dmypy.json
-
-# Pyre type checker
-.pyre/
-
-# 
-.DS_Store
-
-# Editors
-.idea
-.vscode

.github/workflows/development-e2e-tests.yml

@@ -0,0 +1,13 @@
+name: Comment-based E2E test
+
+on:
+  pull_request_review_comment:
+    types: [created]
+
+jobs:
+  e2e:
+    runs-on: ubuntu-18.04
+    steps:
+      - name: logging
+        if: github.event.comment.body == '/run e2e'
+        run: echo "$GITHUB_REF"

.github/workflows/development.yml

@@ -0,0 +1,57 @@
+name: Lint and test (development)
+
+on:
+  pull_request:
+    branches:
+      - development
+jobs:
+  development:
+    runs-on: ubuntu-18.04
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v2
+
+      - name: Setup Python 3.6
+        uses: actions/setup-python@v1
+        with:
+          python-version: 3.6
+
+      - name: Cache pip packages
+        uses: actions/cache@v1
+        with:
+          path: ~/.cache/pip
+          key: pip-dev-${{ hashFiles('setup.py') }}
+
+      - name: Install dependencies
+        run: |
+          python -m pip install --upgrade pip
+          python -m pip install --upgrade setuptools
+          pip install -e .[dev]
+
+      - name: black
+        run: |
+          black --line-length 100 --check setup.py xain_fl tests
+
+      - name: isort
+        run: |
+          isort --check-only --indent=4 -rc setup.py xain_fl tests
+
+      - name: pylint
+        run: |
+          pylint --rcfile=.pylintrc xain_fl tests
+
+      - name: mypy
+        continue-on-error: true
+        run: |
+          mypy xain_fl tests
+
+      - name: Sphinx check
+        env:
+          SPHINXOPTS: "-W"
+        run: |
+          cd docs/
+          make docs
+
+      - name: PyTest
+        run: |
+          pytest -v

.github/workflows/main.yml → .github/workflows/master.yml

@@ -1,4 +1,4 @@
-name: Lint and test
+name: Lint and test (master)
 
 on:
   pull_request:
@@ -20,7 +20,7 @@ jobs:
         uses: actions/cache@v1
         with:
           path: ~/.cache/pip
-          key: pip-${{ hashFiles('setup.py') }}
+          key: pip-master-${{ hashFiles('setup.py') }}
 
       - name: Install dependencies
         run: |
@@ -30,22 +30,20 @@ jobs:
 
       - name: black
         run: |
-          black --check --exclude "xain_fl/cproto/.*_pb2.*" \
-          setup.py conftest.py benchmarks examples xain_fl
+          black --line-length 100 --check setup.py xain_fl tests
 
       - name: isort
         run: |
-          isort --check-only --indent=4 -rc setup.py conftest.py benchmarks \
-          examples xain_fl
+          isort --check-only --indent=4 -rc setup.py xain_fl tests
 
       - name: pylint
         run: |
-          pylint --rcfile=pylint.ini benchmarks examples xain_fl
+          pylint --rcfile=.pylintrc xain_fl tests
 
       - name: mypy
         continue-on-error: true
         run: |
-          mypy benchmarks examples/* xain_fl
+          mypy xain_fl tests
 
       - name: Sphinx check
         env:

.gitignore

@@ -1,13 +1,3 @@
-# Project specific files and directories
-config.cfg
-output/*
-results
-
-# Pantsbuild
-.pants.d
-.pants.workdir.file_lock
-.pants.workdir.file_lock.lock_message
-
 # Byte-compiled / optimized / DLL files
 __pycache__/
 *.py[cod]

.isort.cfg

@@ -1,7 +1,8 @@
 [settings]
-multi_line_output=3
-include_trailing_comma=True
 force_grid_wrap=0
+force_sort_within_sections=True
+include_trailing_comma=True
+known_third_party=xain_proto,xain_sdk
+line_length=100
+multi_line_output=3
 use_parentheses=True
-line_length=88
-skip_glob=*_pb2*

.pylintrc

@@ -0,0 +1,19 @@
+[MESSAGES CONTROL]
+
+disable=bad-continuation,redefined-builtin,duplicate-code,fixme
+
+[TYPECHECK]
+
+# List of members which are set dynamically and missed by pylint inference
+# system, and so shouldn't trigger E1101 when accessed. Python regular
+# expressions are accepted.
+# See: https://github.com/pytorch/pytorch/issues/701#issuecomment-378048755
+generated-members=numpy.*,torch.*
+
+# ignore auto generated grpc files
+ignore-patterns=
+
+[MISCELLANEOUS]
+
+# List of note tags to take in consideration, separated by a comma.
+notes=TODO,FIXME

.readthedocs.yml

@@ -14,4 +14,4 @@ python:
       - method: pip
         path: .
         extra_requirements:
-           - docs
+           - docs