Make it feasible for annotation whether to add an extra line...

Make it feasible for annotation whether to add an extra line "nni.get_next_parameter()" or not (#526)

* fix bug

* add docs

* add ut

* add ut

* add to ci

* update doc

* update doc

* update ut

* add ut to ci

* add ut to ci

* add ut to ci

* add ut to ci

* add ut to ci

* add ut to ci

* add ut to ci

* add ut to ci

* test

* test

* test

* test

* test

* test

* test

* test

* test

* test

* revert

* refactor

* refactor

* s

* merge

* fix annotation for extra line

* add deprecation warning

src/sdk/pynni/nni/smartparam.py

@@ -128,4 +128,6 @@ else:
         if name is None:
             name = '__line{:d}'.format(lineno)
         key = '{}/{}/{}'.format(module, name, func)
+        if trial._params is None:
+            trial.get_next_parameter()
         return trial.get_current_parameter(key)

tools/nni_annotation/code_generator.py

@@ -20,8 +20,8 @@
 
 
 import ast
-
 import astor
+from nni_cmd.common_utils import print_warning
 
 # pylint: disable=unidiomatic-typecheck
 
@@ -218,6 +218,10 @@ class Transformer(ast.NodeTransformer):
         else:
             return node  # not an annotation, ignore it
 
+        if string.startswith('@nni.get_next_parameter('):
+            deprecated_message = "'@nni.get_next_parameter' is deprecated in annotation due to inconvenience. Please remove this line in the trial code."
+            print_warning(deprecated_message)
+
         if string.startswith('@nni.report_intermediate_result(')  \
                 or string.startswith('@nni.report_final_result(') \
                 or string.startswith('@nni.get_next_parameter('):