Remove dead code and pretify some things.

devtools/travis-ci/configure_tests.sh

@@ -7,12 +7,12 @@ if [[ -z ${SRC_DIR} || \
       -z ${SCRIPTS_DIR} || \
       -z ${TEST_DIR} ]] ; then
     echo 'Required environment variable not set!'
-    exit -1
+    exit
 fi
 
 if [[ ${SRC_DIR} = ${TEST_DIR} ]] ; then
     echo "SRC_DIR and TEST_DIR must be different!"
-    exit -1
+    exit
 fi
 
 

elaspic/helper.py

@@ -82,7 +82,7 @@ def make_tarfile(source_dir, output_filename):
 
 # %% Helper functions for sql objects
 def decode_domain_def(domains, merge=True, return_string=False):
-    """ Unlike split_domain(), this function returns a tuple of tuples of strings,
+    """Unlike split_domain(), this function returns a tuple of tuples of strings,
     preserving letter numbering (e.g. 10B)
     """
     if not domains:

elaspic/machine_learning.py

@@ -10,7 +10,6 @@
 from . import helper
 
 
-# %%
 def write_row_to_file(results, output_filename):
     """
     TODO: Add a datetime column to each written row.
@@ -65,8 +64,7 @@ def cross_validate_predictor(data, features, clf_options, output_filename=None):
 
 
 def get_final_predictor(data, features, options):
-    """Train a predictor using the entire dataset.
-    """
+    """Train a predictor using the entire dataset."""
     # Keep only recognized options
     import inspect
     accepted_options = inspect.getargspec(ensemble.GradientBoostingRegressor)[0]