Remove hpo methods (aws#39)

Author: ChoiByungWook

src/sagemaker/amazon/hyperparameter.py

@@ -12,8 +12,6 @@
 # language governing permissions and limitations under the License.
 from __future__ import absolute_import
 
-from sagemaker.tuner import _ParameterRange
-
 
 class Hyperparameter(object):
     """An algorithm hyperparameter with optional validation. Implemented as a python
@@ -48,35 +46,12 @@ def validate(self, value):
                 raise ValueError(error_message)
 
     def __get__(self, obj, objtype):
-        """Return the value of this hyperparameter, whether it be a range or one discrete value"""
-        if '_parameter_ranges' in dir(obj) and self.name in obj._parameter_ranges:
-            return obj._parameter_ranges[self.name]
         if '_hyperparameters' not in dir(obj) or self.name not in obj._hyperparameters:
             raise AttributeError()
         return obj._hyperparameters[self.name]
 
     def __set__(self, obj, value):
-        """Assign values for hyperparameters"""
-        if isinstance(value, _ParameterRange):
-            self._set_parameter_range(obj, value)
-        else:
-            self._set_hyperparameter(obj, value)
-
-    def _set_parameter_range(self, obj, value):
-        # remove from the hyperparameters if it's there
-        if '_hyperparameters' in dir(obj) and self.name in obj._hyperparameters:
-            del obj._hyperparameters[self.name]
-
-        if '_parameter_ranges' not in dir(obj):
-            obj._parameter_ranges = dict()
-        obj._parameter_ranges[self.name] = value
-
-    def _set_hyperparameter(self, obj, value):
         """Validate the supplied value and set this hyperparameter to value"""
-        # remove from the parameter_ranges if it's there
-        if '_parameter_ranges' in dir(obj) and self.name in obj._parameter_ranges:
-            del obj._parameter_ranges[self.name]
-
         value = None if value is None else self.data_type(value)
         self.validate(value)
         if '_hyperparameters' not in dir(obj):
@@ -85,10 +60,7 @@ def _set_hyperparameter(self, obj, value):
 
     def __delete__(self, obj):
         """Delete this hyperparameter"""
-        if '_hyperparameters' in dir(obj) and self.name in obj._hyperparameters:
-            del obj._hyperparameters[self.name]
-        if '_parameter_ranges' in dir(obj) and self.name in obj._parameter_ranges:
-            del obj._parameter_ranges[self.name]
+        del obj._hyperparameters[self.name]
 
     @staticmethod
     def serialize_all(obj):

tests/unit/test_hyperparameter.py

@@ -14,7 +14,6 @@
 
 import pytest
 from sagemaker.amazon.hyperparameter import Hyperparameter
-from sagemaker.tuner import ContinuousParameter
 
 
 class Test(object):
@@ -75,12 +74,3 @@ def test_from_string():
 
     x.validated = from_api
     assert x.validated == value
-
-
-def test_delete_parameter_range():
-    x = Test()
-    x.validated = ContinuousParameter(0, 5)
-    del(x.validated)
-
-    with pytest.raises(AttributeError):
-        x.validated