Upd base by arodiss · Pull Request #1 · arodiss/sagemaker-python-sdk

arodiss · 2019-07-25T12:53:00Z

Issue #, if available:

Description of changes:

Merge Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.

I have read the CONTRIBUTING doc
I used the commit message format described in CONTRIBUTING
I have added tests that prove my fix is effective or that my feature works (if appropriate)
I have updated any necessary documentation (if appropriate)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

By running Pylint before any of the unit tests (and dependency installs), the import-error check will always fail since the dependencies are not yet installed. We could move Pylint to a later stage to resolve this, but there's value in this quick check occurring before the unit tests. As a result, this Pylint check is being disabled.

add authentication info

* add functions, tests and doc for CodeCommit

Note that this commit also raises ValueErrors in situations that would previously have returned None. Per PEP8: Be consistent in return statements. Either all return statements in a function should return an expression, or none of them should. If any return statement returns an expression, any return statements where no value is returned should explicitly state this as return None, and an explicit return statement should be present at the end of the function (if reachable).

Note that this commit will also enable simplifiable-if-statement, as there are no code changes needed for it.

The logic behind this rule is to improve readability by defining all the attributes of a class inside the init function, even if it simply sets them to None.

Per PEP8: Imports should be grouped in the following order: 1- Standard library imports. 2- Related third party imports. 3- Local application/library specific imports.

* change: enable wrong-import-order Pylint check Per PEP8: Imports should be grouped in the following order: 1- Standard library imports. 2- Related third party imports. 3- Local application/library specific imports.

* change: enable consider-using-in Pylint check

* change: enable too-many-public-methods Pylint check This is a useful check to have, but is a lot of work to retroactively enforce. Enabling it while ignoring the single violation allows the validation to run for future code.

This commit will add an exclusion for all auto-generated files. I chose to ignore the single violation, because the alternative is confusingly convoluted: `(hasattr(obj, '__getitem__') if hasattr(obj, '__iter__') else isinstance(obj, str))`

The check recommendations are only valid for packages that exclusively support Python 3. The changes cannot be made in Python 2. The TODO was updated to clarify this.

This commit will format all existing docstring to follow Google style: https://sphinxcontrib-napoleon.readthedocs.io/en/latest/example_google.html This commit will also add docstring placeholders to any class or method previously missing it. An ideal approach would be to take the time to include meaningful docstrings in every file. However, since that is not a task that will be prioritized, I've declared docstring bankruptcy on this package, in order to enforce docstring on all future code changes to this package.

…er() with frameworks (#944)

[pr-827][followups] Improve documentation of some functions. Also some unit test fixes. See comments from marcio in #827

Added instructions that allow for a low-cost ~10min environment setup.

arodiss · 2019-07-25T12:53:00Z

Issue #, if available:

Description of changes:

Merge Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.

I have read the CONTRIBUTING doc
I used the commit message format described in CONTRIBUTING
I have added tests that prove my fix is effective or that my feature works (if appropriate)
I have updated any necessary documentation (if appropriate)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

By running Pylint before any of the unit tests (and dependency installs), the import-error check will always fail since the dependencies are not yet installed. We could move Pylint to a later stage to resolve this, but there's value in this quick check occurring before the unit tests. As a result, this Pylint check is being disabled.

add authentication info

* add functions, tests and doc for CodeCommit

Note that this commit also raises ValueErrors in situations that would previously have returned None. Per PEP8: Be consistent in return statements. Either all return statements in a function should return an expression, or none of them should. If any return statement returns an expression, any return statements where no value is returned should explicitly state this as return None, and an explicit return statement should be present at the end of the function (if reachable).

Note that this commit will also enable simplifiable-if-statement, as there are no code changes needed for it.

The logic behind this rule is to improve readability by defining all the attributes of a class inside the init function, even if it simply sets them to None.

Per PEP8: Imports should be grouped in the following order: 1- Standard library imports. 2- Related third party imports. 3- Local application/library specific imports.

* change: enable wrong-import-order Pylint check Per PEP8: Imports should be grouped in the following order: 1- Standard library imports. 2- Related third party imports. 3- Local application/library specific imports.

* change: enable consider-using-in Pylint check

* change: enable too-many-public-methods Pylint check This is a useful check to have, but is a lot of work to retroactively enforce. Enabling it while ignoring the single violation allows the validation to run for future code.

This commit will add an exclusion for all auto-generated files. I chose to ignore the single violation, because the alternative is confusingly convoluted: `(hasattr(obj, '__getitem__') if hasattr(obj, '__iter__') else isinstance(obj, str))`

The check recommendations are only valid for packages that exclusively support Python 3. The changes cannot be made in Python 2. The TODO was updated to clarify this.

This commit will format all existing docstring to follow Google style: https://sphinxcontrib-napoleon.readthedocs.io/en/latest/example_google.html This commit will also add docstring placeholders to any class or method previously missing it. An ideal approach would be to take the time to include meaningful docstrings in every file. However, since that is not a task that will be prioritized, I've declared docstring bankruptcy on this package, in order to enforce docstring on all future code changes to this package.

…er() with frameworks (#944)

[pr-827][followups] Improve documentation of some functions. Also some unit test fixes. See comments from marcio in #827

Added instructions that allow for a low-cost ~10min environment setup.

arodiss · 2019-07-25T12:53:00Z

Issue #, if available:

Description of changes:

Merge Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.

I have read the CONTRIBUTING doc
I used the commit message format described in CONTRIBUTING
I have added tests that prove my fix is effective or that my feature works (if appropriate)
I have updated any necessary documentation (if appropriate)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

By running Pylint before any of the unit tests (and dependency installs), the import-error check will always fail since the dependencies are not yet installed. We could move Pylint to a later stage to resolve this, but there's value in this quick check occurring before the unit tests. As a result, this Pylint check is being disabled.

add authentication info

* add functions, tests and doc for CodeCommit

Note that this commit also raises ValueErrors in situations that would previously have returned None. Per PEP8: Be consistent in return statements. Either all return statements in a function should return an expression, or none of them should. If any return statement returns an expression, any return statements where no value is returned should explicitly state this as return None, and an explicit return statement should be present at the end of the function (if reachable).

Note that this commit will also enable simplifiable-if-statement, as there are no code changes needed for it.

The logic behind this rule is to improve readability by defining all the attributes of a class inside the init function, even if it simply sets them to None.

Per PEP8: Imports should be grouped in the following order: 1- Standard library imports. 2- Related third party imports. 3- Local application/library specific imports.

* change: enable wrong-import-order Pylint check Per PEP8: Imports should be grouped in the following order: 1- Standard library imports. 2- Related third party imports. 3- Local application/library specific imports.

* change: enable consider-using-in Pylint check

* change: enable too-many-public-methods Pylint check This is a useful check to have, but is a lot of work to retroactively enforce. Enabling it while ignoring the single violation allows the validation to run for future code.

This commit will add an exclusion for all auto-generated files. I chose to ignore the single violation, because the alternative is confusingly convoluted: `(hasattr(obj, '__getitem__') if hasattr(obj, '__iter__') else isinstance(obj, str))`

The check recommendations are only valid for packages that exclusively support Python 3. The changes cannot be made in Python 2. The TODO was updated to clarify this.

This commit will format all existing docstring to follow Google style: https://sphinxcontrib-napoleon.readthedocs.io/en/latest/example_google.html This commit will also add docstring placeholders to any class or method previously missing it. An ideal approach would be to take the time to include meaningful docstrings in every file. However, since that is not a task that will be prioritized, I've declared docstring bankruptcy on this package, in order to enforce docstring on all future code changes to this package.

…er() with frameworks (#944)

[pr-827][followups] Improve documentation of some functions. Also some unit test fixes. See comments from marcio in #827

Added instructions that allow for a low-cost ~10min environment setup.

arodiss · 2019-07-25T12:53:00Z

Issue #, if available:

Description of changes:

Merge Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.

I have read the CONTRIBUTING doc
I used the commit message format described in CONTRIBUTING
I have added tests that prove my fix is effective or that my feature works (if appropriate)
I have updated any necessary documentation (if appropriate)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

By running Pylint before any of the unit tests (and dependency installs), the import-error check will always fail since the dependencies are not yet installed. We could move Pylint to a later stage to resolve this, but there's value in this quick check occurring before the unit tests. As a result, this Pylint check is being disabled.

add authentication info

* add functions, tests and doc for CodeCommit

Note that this commit also raises ValueErrors in situations that would previously have returned None. Per PEP8: Be consistent in return statements. Either all return statements in a function should return an expression, or none of them should. If any return statement returns an expression, any return statements where no value is returned should explicitly state this as return None, and an explicit return statement should be present at the end of the function (if reachable).

Note that this commit will also enable simplifiable-if-statement, as there are no code changes needed for it.

The logic behind this rule is to improve readability by defining all the attributes of a class inside the init function, even if it simply sets them to None.

Per PEP8: Imports should be grouped in the following order: 1- Standard library imports. 2- Related third party imports. 3- Local application/library specific imports.

* change: enable wrong-import-order Pylint check Per PEP8: Imports should be grouped in the following order: 1- Standard library imports. 2- Related third party imports. 3- Local application/library specific imports.

* change: enable consider-using-in Pylint check

* change: enable too-many-public-methods Pylint check This is a useful check to have, but is a lot of work to retroactively enforce. Enabling it while ignoring the single violation allows the validation to run for future code.

This commit will add an exclusion for all auto-generated files. I chose to ignore the single violation, because the alternative is confusingly convoluted: `(hasattr(obj, '__getitem__') if hasattr(obj, '__iter__') else isinstance(obj, str))`

The check recommendations are only valid for packages that exclusively support Python 3. The changes cannot be made in Python 2. The TODO was updated to clarify this.

This commit will format all existing docstring to follow Google style: https://sphinxcontrib-napoleon.readthedocs.io/en/latest/example_google.html This commit will also add docstring placeholders to any class or method previously missing it. An ideal approach would be to take the time to include meaningful docstrings in every file. However, since that is not a task that will be prioritized, I've declared docstring bankruptcy on this package, in order to enforce docstring on all future code changes to this package.

…er() with frameworks (#944)

[pr-827][followups] Improve documentation of some functions. Also some unit test fixes. See comments from marcio in #827

Added instructions that allow for a low-cost ~10min environment setup.

arodiss · 2019-07-25T12:53:00Z

Issue #, if available:

Description of changes:

Merge Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.

I have read the CONTRIBUTING doc
I used the commit message format described in CONTRIBUTING
I have added tests that prove my fix is effective or that my feature works (if appropriate)
I have updated any necessary documentation (if appropriate)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

By running Pylint before any of the unit tests (and dependency installs), the import-error check will always fail since the dependencies are not yet installed. We could move Pylint to a later stage to resolve this, but there's value in this quick check occurring before the unit tests. As a result, this Pylint check is being disabled.

add authentication info

* add functions, tests and doc for CodeCommit

Note that this commit also raises ValueErrors in situations that would previously have returned None. Per PEP8: Be consistent in return statements. Either all return statements in a function should return an expression, or none of them should. If any return statement returns an expression, any return statements where no value is returned should explicitly state this as return None, and an explicit return statement should be present at the end of the function (if reachable).

Note that this commit will also enable simplifiable-if-statement, as there are no code changes needed for it.

The logic behind this rule is to improve readability by defining all the attributes of a class inside the init function, even if it simply sets them to None.

Per PEP8: Imports should be grouped in the following order: 1- Standard library imports. 2- Related third party imports. 3- Local application/library specific imports.

* change: enable wrong-import-order Pylint check Per PEP8: Imports should be grouped in the following order: 1- Standard library imports. 2- Related third party imports. 3- Local application/library specific imports.

* change: enable consider-using-in Pylint check

* change: enable too-many-public-methods Pylint check This is a useful check to have, but is a lot of work to retroactively enforce. Enabling it while ignoring the single violation allows the validation to run for future code.

This commit will add an exclusion for all auto-generated files. I chose to ignore the single violation, because the alternative is confusingly convoluted: `(hasattr(obj, '__getitem__') if hasattr(obj, '__iter__') else isinstance(obj, str))`

The check recommendations are only valid for packages that exclusively support Python 3. The changes cannot be made in Python 2. The TODO was updated to clarify this.

This commit will format all existing docstring to follow Google style: https://sphinxcontrib-napoleon.readthedocs.io/en/latest/example_google.html This commit will also add docstring placeholders to any class or method previously missing it. An ideal approach would be to take the time to include meaningful docstrings in every file. However, since that is not a task that will be prioritized, I've declared docstring bankruptcy on this package, in order to enforce docstring on all future code changes to this package.

…er() with frameworks (#944)

[pr-827][followups] Improve documentation of some functions. Also some unit test fixes. See comments from marcio in #827

Added instructions that allow for a low-cost ~10min environment setup.

arodiss · 2019-07-25T12:53:00Z

Issue #, if available:

Description of changes:

Merge Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.

I have read the CONTRIBUTING doc
I used the commit message format described in CONTRIBUTING
I have added tests that prove my fix is effective or that my feature works (if appropriate)
I have updated any necessary documentation (if appropriate)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

By running Pylint before any of the unit tests (and dependency installs), the import-error check will always fail since the dependencies are not yet installed. We could move Pylint to a later stage to resolve this, but there's value in this quick check occurring before the unit tests. As a result, this Pylint check is being disabled.

add authentication info

* add functions, tests and doc for CodeCommit

Note that this commit also raises ValueErrors in situations that would previously have returned None. Per PEP8: Be consistent in return statements. Either all return statements in a function should return an expression, or none of them should. If any return statement returns an expression, any return statements where no value is returned should explicitly state this as return None, and an explicit return statement should be present at the end of the function (if reachable).

Note that this commit will also enable simplifiable-if-statement, as there are no code changes needed for it.

The logic behind this rule is to improve readability by defining all the attributes of a class inside the init function, even if it simply sets them to None.

Per PEP8: Imports should be grouped in the following order: 1- Standard library imports. 2- Related third party imports. 3- Local application/library specific imports.

* change: enable wrong-import-order Pylint check Per PEP8: Imports should be grouped in the following order: 1- Standard library imports. 2- Related third party imports. 3- Local application/library specific imports.

* change: enable consider-using-in Pylint check

* change: enable too-many-public-methods Pylint check This is a useful check to have, but is a lot of work to retroactively enforce. Enabling it while ignoring the single violation allows the validation to run for future code.

This commit will add an exclusion for all auto-generated files. I chose to ignore the single violation, because the alternative is confusingly convoluted: `(hasattr(obj, '__getitem__') if hasattr(obj, '__iter__') else isinstance(obj, str))`

The check recommendations are only valid for packages that exclusively support Python 3. The changes cannot be made in Python 2. The TODO was updated to clarify this.

This commit will format all existing docstring to follow Google style: https://sphinxcontrib-napoleon.readthedocs.io/en/latest/example_google.html This commit will also add docstring placeholders to any class or method previously missing it. An ideal approach would be to take the time to include meaningful docstrings in every file. However, since that is not a task that will be prioritized, I've declared docstring bankruptcy on this package, in order to enforce docstring on all future code changes to this package.

…er() with frameworks (#944)

[pr-827][followups] Improve documentation of some functions. Also some unit test fixes. See comments from marcio in #827

Added instructions that allow for a low-cost ~10min environment setup.

arodiss · 2019-07-25T12:53:00Z

Issue #, if available:

Description of changes:

Merge Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.

I have read the CONTRIBUTING doc
I used the commit message format described in CONTRIBUTING
I have added tests that prove my fix is effective or that my feature works (if appropriate)
I have updated any necessary documentation (if appropriate)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

By running Pylint before any of the unit tests (and dependency installs), the import-error check will always fail since the dependencies are not yet installed. We could move Pylint to a later stage to resolve this, but there's value in this quick check occurring before the unit tests. As a result, this Pylint check is being disabled.

add authentication info

* add functions, tests and doc for CodeCommit

Note that this commit also raises ValueErrors in situations that would previously have returned None. Per PEP8: Be consistent in return statements. Either all return statements in a function should return an expression, or none of them should. If any return statement returns an expression, any return statements where no value is returned should explicitly state this as return None, and an explicit return statement should be present at the end of the function (if reachable).

Note that this commit will also enable simplifiable-if-statement, as there are no code changes needed for it.

The logic behind this rule is to improve readability by defining all the attributes of a class inside the init function, even if it simply sets them to None.

Per PEP8: Imports should be grouped in the following order: 1- Standard library imports. 2- Related third party imports. 3- Local application/library specific imports.

* change: enable wrong-import-order Pylint check Per PEP8: Imports should be grouped in the following order: 1- Standard library imports. 2- Related third party imports. 3- Local application/library specific imports.

* change: enable consider-using-in Pylint check

* change: enable too-many-public-methods Pylint check This is a useful check to have, but is a lot of work to retroactively enforce. Enabling it while ignoring the single violation allows the validation to run for future code.

This commit will add an exclusion for all auto-generated files. I chose to ignore the single violation, because the alternative is confusingly convoluted: `(hasattr(obj, '__getitem__') if hasattr(obj, '__iter__') else isinstance(obj, str))`

The check recommendations are only valid for packages that exclusively support Python 3. The changes cannot be made in Python 2. The TODO was updated to clarify this.

This commit will format all existing docstring to follow Google style: https://sphinxcontrib-napoleon.readthedocs.io/en/latest/example_google.html This commit will also add docstring placeholders to any class or method previously missing it. An ideal approach would be to take the time to include meaningful docstrings in every file. However, since that is not a task that will be prioritized, I've declared docstring bankruptcy on this package, in order to enforce docstring on all future code changes to this package.

…er() with frameworks (#944)

[pr-827][followups] Improve documentation of some functions. Also some unit test fixes. See comments from marcio in #827

Added instructions that allow for a low-cost ~10min environment setup.

arodiss · 2019-07-25T12:53:00Z

Issue #, if available:

Description of changes:

Merge Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.

I have read the CONTRIBUTING doc
I used the commit message format described in CONTRIBUTING
I have added tests that prove my fix is effective or that my feature works (if appropriate)
I have updated any necessary documentation (if appropriate)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

By running Pylint before any of the unit tests (and dependency installs), the import-error check will always fail since the dependencies are not yet installed. We could move Pylint to a later stage to resolve this, but there's value in this quick check occurring before the unit tests. As a result, this Pylint check is being disabled.

add authentication info

* add functions, tests and doc for CodeCommit

Note that this commit also raises ValueErrors in situations that would previously have returned None. Per PEP8: Be consistent in return statements. Either all return statements in a function should return an expression, or none of them should. If any return statement returns an expression, any return statements where no value is returned should explicitly state this as return None, and an explicit return statement should be present at the end of the function (if reachable).

Note that this commit will also enable simplifiable-if-statement, as there are no code changes needed for it.

The logic behind this rule is to improve readability by defining all the attributes of a class inside the init function, even if it simply sets them to None.

Per PEP8: Imports should be grouped in the following order: 1- Standard library imports. 2- Related third party imports. 3- Local application/library specific imports.

* change: enable wrong-import-order Pylint check Per PEP8: Imports should be grouped in the following order: 1- Standard library imports. 2- Related third party imports. 3- Local application/library specific imports.

* change: enable consider-using-in Pylint check

* change: enable too-many-public-methods Pylint check This is a useful check to have, but is a lot of work to retroactively enforce. Enabling it while ignoring the single violation allows the validation to run for future code.

This commit will add an exclusion for all auto-generated files. I chose to ignore the single violation, because the alternative is confusingly convoluted: `(hasattr(obj, '__getitem__') if hasattr(obj, '__iter__') else isinstance(obj, str))`

The check recommendations are only valid for packages that exclusively support Python 3. The changes cannot be made in Python 2. The TODO was updated to clarify this.

This commit will format all existing docstring to follow Google style: https://sphinxcontrib-napoleon.readthedocs.io/en/latest/example_google.html This commit will also add docstring placeholders to any class or method previously missing it. An ideal approach would be to take the time to include meaningful docstrings in every file. However, since that is not a task that will be prioritized, I've declared docstring bankruptcy on this package, in order to enforce docstring on all future code changes to this package.

…er() with frameworks (#944)

[pr-827][followups] Improve documentation of some functions. Also some unit test fixes. See comments from marcio in #827

Added instructions that allow for a low-cost ~10min environment setup.

knakadand others added 30 commits July 10, 2019 09:11

prepare release v1.33.0
7acfb31

update development version to v1.33.1.dev0
1705b13

change: enable unidiomatic-typecheck pylint check (#921)
b1f95fb

change: enable no-else-return and no-else-raise pylint checks (#925)
0cf5902

change: fix list serialization for 1P algos (#922)
100d906

change: enable simplifiable-if-expression pylint checks (#926)
6061a8b

feature: deal with credentials for Git support for GitHub (#914)
fb309bc
add authentication info

feature: Git integration for CodeCommit (#927)
2cf69b2
* add functions, tests and doc for CodeCommit

change: enable consider-merging-isinstance Pylint check (#932)
1b4c2b1
Note that this commit will also enable simplifiable-if-statement, as there are no code changes needed for it.

change: enable attribute-defined-outside-init Pylint check (#933)
798299a
The logic behind this rule is to improve readability by defining all the attributes of a class inside the init function, even if it simply sets them to None.

change: enable wrong-import-order Pylint check (#935)
b6b3cf6
Per PEP8: Imports should be grouped in the following order: 1- Standard library imports. 2- Related third party imports. 3- Local application/library specific imports.

change: enable ungrouped-imports Pylint check (#936)
6adb29b
* change: enable wrong-import-order Pylint check Per PEP8: Imports should be grouped in the following order: 1- Standard library imports. 2- Related third party imports. 3- Local application/library specific imports.

change: fix attach for 1P algorithm estimators (#931)
3734314

change: set num_processes_per_host only if provided by user (#928)
1badf76

change: enable consider-using-in Pylint check (#938)
aab1675
* change: enable consider-using-in Pylint check

change: enable too-many-public-methods Pylint check (#939)
39adbbc
* change: enable too-many-public-methods Pylint check This is a useful check to have, but is a lot of work to retroactively enforce. Enabling it while ignoring the single violation allows the validation to run for future code.

change: enable chained-comparison Pylint check (#940)
36bbf8a

change: modify TODO on disabled Pylint check (#943)
91c5529
The check recommendations are only valid for packages that exclusively support Python 3. The changes cannot be made in Python 2. The TODO was updated to clarify this.

prepare release v1.34.0
8871a14

update development version to v1.34.1.dev0
87d455a

change: add MXNet 1.4.1 support (#886)
40f1f98

change: allow serving script to be defined for deploy() and transform…
2dfe7d3
…er() with frameworks (#944)

change: update PyTorch version (#947)
910d9f8

change: improve documentation of some functions (#864)
946adeb
[pr-827][followups] Improve documentation of some functions. Also some unit test fixes. See comments from marcio in #827

doc: update using_tensorflow topic (#946)
af4b3e3

fix: update TensorFlow script mode dependency list (#869)
7b9ad5c

mvsuspand others added 5 commits July 22, 2019 10:33

change: improving Chainer integ tests (#872)
d3c5205

change: enable line-too-long Pylint check (#948)
4ebb088

doc: add instructions for setting up Cloud9 environment. (#949)
d3892fb
Added instructions that allow for a low-cost ~10min environment setup.

prepare release v1.34.1
6545cf8

update development version to v1.34.2.dev0
dfa9644

arodiss merged commit d7d33c4 into arodiss:masterJul 25, 2019