Shuffling has been removed

dshahid380 · dshahid380 · commit 32a0f98a4048 · 2019-04-12T22:41:18.000+05:30
diff --git a/learning.py b/learning.py
@@ -1050,33 +1050,25 @@ def grade_learner(predict, tests):
 
 
 def train_test_split(dataset, start = None, end = None, test_split = None):
-    """If you are giving 'start' and 'end' as a parameter,
-    then it will return testing set from index 'start' to 'end'
-    and rest for training.
-    If you give 'test_split' as parameter then it will first shuffle the
-    dataset then return test_split * 100% as testing set and rest as 
+    """If you are giving 'start' and 'end' as parameters,
+    then it will return the testing set from index 'start' to 'end'
+    and the rest for training.
+    If you give 'test_split' as a parameter then it will return 
+    test_split * 100% as the testing set and the rest as 
     training set.
     """
-    
-    if start == None and end != None:
-        raise ValueError("'start' parameter is missing")
-
-    if start != None and end == None:
-        raise ValueError("'end' parameter is missing")
-
+    examples = dataset.examples
     if test_split == None:
-        examples = dataset.examples
         train = examples[:start] + examples[end:]
         val = examples[start:end]
-        return train, val
     else:
-        examples = dataset.examples
         total_size = len(examples)
         val_size = int(total_size * test_split)
         train_size = total_size - val_size
         train = examples[:train_size]
         val = examples[train_size:total_size]
-        return train, val
+
+    return train, val
 
 
 def cross_validation(learner, size, dataset, k=10, trials=1):
