From 5103239d6ed9ed0c024e574dd9611d2dbb34b25c Mon Sep 17 00:00:00 2001 From: Germano Gabbianelli Date: Thu, 13 Dec 2018 16:03:19 +0100 Subject: [PATCH] Fix __all__ in sagemaker.rl __all__ should be a list of names (str), not objects. Moreover it "should be placed after the module docstring but before any import statements except from __future__ imports" (see PEP8) --- src/sagemaker/rl/__init__.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/sagemaker/rl/__init__.py b/src/sagemaker/rl/__init__.py index 62a0c1e934..562cf863ed 100644 --- a/src/sagemaker/rl/__init__.py +++ b/src/sagemaker/rl/__init__.py @@ -12,7 +12,8 @@ # language governing permissions and limitations under the License. from __future__ import absolute_import +__all__ = [ + 'RLEstimator', 'RLToolkit', 'RLFramework', 'TOOLKIT_FRAMEWORK_VERSION_MAP'] + from sagemaker.rl.estimator import (RLEstimator, RLToolkit, RLFramework, TOOLKIT_FRAMEWORK_VERSION_MAP) - -__all__ = [RLEstimator, RLToolkit, RLFramework, TOOLKIT_FRAMEWORK_VERSION_MAP]