Refactor batch parley, prepare for MultiAgent Batch Parley (facebookr…

…esearch#29)

* start

* batch cleanup, prepare for multiagent batch

* oops

* generalize display

* cleaning single vs. multi + comments

* cleaning single vs. multi + comments

* cleaning single vs. multi + comments

* small update

* small fixes

* Tasks (facebookresearch#30)

Task fixes, changes multiworld.display a tiny bit, added a few init.py's, and fixed a few docs.

* multiagent batch

* start

* batch cleanup, prepare for multiagent batch

* oops

* generalize display

* cleaning single vs. multi + comments

* cleaning single vs. multi + comments

* cleaning single vs. multi + comments

* small update

* small fixes

* multiagent batch

* small

* bug

parlai/core/fbdialog_teacher.py

@@ -51,12 +51,12 @@ def __init__(self, opt, shared=None):
 
     def label_candidates(self):
         return self.cands
-
+    
     def load_cands(self, path):
-    """Load global fixed set of candidate labels that the teacher provides every
-    example (the true labels for a specific example are also added to this set,
-    so that it's possible to get the right answer).
-    """
+        """Load global fixed set of candidate labels that the teacher provides every
+        example (the true labels for a specific example are also added to this set,
+        so that it's possible to get the right answer).
+        """
         if path is None:
             return None
         cands = []

parlai/core/params.py

@@ -14,6 +14,11 @@
 def str2bool(value):
     return value.lower() in ('yes', 'true', 't', '1', 'y')
 
+def path(s):
+    # Add a trailing slash if its not there.
+    if s[-1] != '/':
+        s += '/'
+    return s
 
 class ParlaiParser(object):
     """Pseudo-extension of argparse which sets a number of parameters for the
@@ -96,10 +101,13 @@ def parse_args(self, args=None, print_args=True):
         """
         self.args = self.parser.parse_args(args=args)
         self.opt = {k: v for k, v in vars(self.args).items() if v is not None}
-        if print_args:
-            self.print_args()
         if 'download_path' in self.opt:
+            self.opt['download_path'] = path(self.opt['download_path'])
             os.environ['PARLAI_DOWNPATH'] = self.opt['download_path']
+        if 'datapath' in self.opt:
+            self.opt['datapath'] = path(self.opt['datapath'])
+        if print_args:
+            self.print_args()
         return self.opt
 
     def print_args(self):