A tiny fix in PyDataProvider2

* hidden decorator kwargs in DataProvider.__init__ * also add unit test for this.
PaddlePaddle · Dec 21, 2016 · 9f5e742 · 9f5e742
1 parent 2965df5
commit 9f5e742
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/paddle/gserver/tests/test_PyDataProvider2.py b/paddle/gserver/tests/test_PyDataProvider2.py
@@ -17,7 +17,7 @@
 from paddle.trainer.PyDataProvider2 import *
 
 
-@provider(input_types=[dense_vector(200, seq_type=SequenceType.NO_SEQUENCE)])
+@provider(slots=[dense_vector(200, seq_type=SequenceType.NO_SEQUENCE)])
 def test_dense_no_seq(setting, filename):
     for i in xrange(200):
         yield [(float(j - 100) * float(i + 1)) / 200.0 for j in xrange(200)]

diff --git a/python/paddle/trainer/PyDataProvider2.py b/python/paddle/trainer/PyDataProvider2.py
@@ -232,7 +232,7 @@ def provider(input_types=None,
              check=False,
              check_fail_continue=False,
              init_hook=None,
-             **kwargs):
+             **outter_kwargs):
     """
     Provider decorator. Use it to make a function into PyDataProvider2 object.
     In this function, user only need to get each sample for some train/test
@@ -318,10 +318,10 @@ def __init__(self, file_list, **kwargs):
                 self.logger = logging.getLogger("")
                 self.logger.setLevel(logging.INFO)
                 self.input_types = None
-                if 'slots' in kwargs:
+                if 'slots' in outter_kwargs:
                     self.logger.warning('setting slots value is deprecated, '
                                         'please use input_types instead.')
-                    self.slots = kwargs['slots']
+                    self.slots = outter_kwargs['slots']
                 self.slots = input_types
                 self.should_shuffle = should_shuffle