Code quality fixes

CHANGES.rst

@@ -1,4 +1,4 @@
-3.0.0 (upcoming)
+3.0.0 (future)
 =================
 
 **Breaking Changes**
@@ -28,7 +28,6 @@ Bug fixes
 - Workaround for GNU Assembler bug causing incorrect calculations [#196]
 
 
-
 2.3.1 (2019-06-21)
 ================
 

Corrfunc/mocks/DDrppi_mocks.py

@@ -328,7 +328,7 @@ def DDrppi_mocks(autocorr, cosmology, nthreads, pimax, binfile,
     import numpy as np
     from Corrfunc.utils import translate_isa_string_to_enum, fix_ra_dec,\
         return_file_with_rbins, convert_to_native_endian,\
-        is_native_endian, sys_pipes, process_weights
+        sys_pipes, process_weights
     from future.utils import bytes_to_native_str
 
     if not autocorr:

Corrfunc/mocks/DDsmu_mocks.py

@@ -248,7 +248,8 @@ def DDsmu_mocks(autocorr, cosmology, nthreads, mu_max, nmu_bins, binfile,
 
     import numpy as np
     from Corrfunc.utils import translate_isa_string_to_enum, fix_ra_dec,\
-        return_file_with_rbins, sys_pipes, process_weights, convert_to_native_endian
+        return_file_with_rbins, convert_to_native_endian,\
+        sys_pipes, process_weights
     from future.utils import bytes_to_native_str
 
     # Check if mu_max is scalar
@@ -260,7 +261,7 @@ def DDsmu_mocks(autocorr, cosmology, nthreads, mu_max, nmu_bins, binfile,
 
     # Check that mu_max is within (0.0, 1.0]
     if mu_max <= 0.0 or mu_max > 1.0:
-        msg = "The parameter `mu_max` = {0}, is the max. of cosine of an "
+        msg = "The parameter `mu_max` = {0}, is the max. of cosine of an "\
         "angle and should be within (0.0, 1.0]".format(mu_max)
         raise ValueError(msg)
 

Corrfunc/mocks/DDtheta_mocks.py

@@ -277,7 +277,7 @@ def DDtheta_mocks(autocorr, nthreads, binfile,
     import numpy as np
     from Corrfunc.utils import translate_isa_string_to_enum, fix_ra_dec,\
         return_file_with_rbins, convert_to_native_endian,\
-        is_native_endian, sys_pipes, process_weights
+        sys_pipes, process_weights
     from future.utils import bytes_to_native_str
 
     if autocorr == 0:

Corrfunc/mocks/vpf_mocks.py

@@ -282,8 +282,7 @@ def vpf_mocks(rmax, nbins, nspheres, numpN,
     import numpy as np
     from future.utils import bytes_to_native_str
     from Corrfunc.utils import translate_isa_string_to_enum,\
-        return_file_with_rbins, convert_to_native_endian,\
-        is_native_endian, sys_pipes, process_weights
+        convert_to_native_endian, sys_pipes
 
     # Ensure all input arrays are native endian
     RA, DEC, CZ, RAND_RA, RAND_DEC, RAND_CZ = [

Corrfunc/theory/DD.py

@@ -171,7 +171,8 @@ def DD(autocorr, nthreads, binfile, X1, Y1, Z1, weights1=None, periodic=True,
     >>> Y = np.random.uniform(0, boxsize, N)
     >>> Z = np.random.uniform(0, boxsize, N)
     >>> weights = np.ones_like(X)
-    >>> results = DD(autocorr, nthreads, binfile, X, Y, Z, weights1=weights, weight_type='pair_product', output_ravg=True)
+    >>> results = DD(autocorr, nthreads, binfile, X, Y, Z, weights1=weights,
+    ...              weight_type='pair_product', output_ravg=True)
     >>> for r in results: print("{0:10.6f} {1:10.6f} {2:10.6f} {3:10d} {4:10.6f}".
     ...                         format(r['rmin'], r['rmax'], r['ravg'],
     ...                         r['npairs'], r['weightavg'])) # doctest: +NORMALIZE_WHITESPACE
@@ -201,7 +202,7 @@ def DD(autocorr, nthreads, binfile, X1, Y1, Z1, weights1=None, periodic=True,
     import numpy as np
     from Corrfunc.utils import translate_isa_string_to_enum,\
         return_file_with_rbins, convert_to_native_endian,\
-        is_native_endian, sys_pipes, process_weights
+        sys_pipes, process_weights
     from future.utils import bytes_to_native_str
 
     if not autocorr:

Corrfunc/theory/DDrppi.py

@@ -247,7 +247,7 @@ def DDrppi(autocorr, nthreads, pimax, binfile, X1, Y1, Z1, weights1=None,
     import numpy as np
     from Corrfunc.utils import translate_isa_string_to_enum,\
         return_file_with_rbins, convert_to_native_endian,\
-        is_native_endian, sys_pipes, process_weights
+        sys_pipes, process_weights
     from future.utils import bytes_to_native_str
 
     if not autocorr:

Corrfunc/theory/DDsmu.py

@@ -254,7 +254,7 @@ def DDsmu(autocorr, nthreads, binfile, mu_max, nmu_bins,
     import numpy as np
     from Corrfunc.utils import translate_isa_string_to_enum,\
         return_file_with_rbins, convert_to_native_endian,\
-        is_native_endian, sys_pipes, process_weights
+        sys_pipes, process_weights
 
     from future.utils import bytes_to_native_str
 
@@ -267,7 +267,7 @@ def DDsmu(autocorr, nthreads, binfile, mu_max, nmu_bins,
 
     # Check that mu_max is within (0.0, 1.0]
     if mu_max <= 0.0 or mu_max > 1.0:
-        msg = "The parameter `mu_max` = {0}, is the max. of cosine of an "
+        msg = "The parameter `mu_max` = {0}, is the max. of cosine of an "\
         "angle and should be within (0.0, 1.0]".format(mu_max)
         raise ValueError(msg)
 

Corrfunc/theory/vpf.py

@@ -189,7 +189,7 @@ def vpf(rmax, nbins, nspheres, numpN, seed,
     import numpy as np
     from future.utils import bytes_to_native_str
     from Corrfunc.utils import translate_isa_string_to_enum,\
-        convert_to_native_endian, is_native_endian, sys_pipes, process_weights
+        convert_to_native_endian, sys_pipes
     from math import pi
 
     if numpN <= 0:

Corrfunc/theory/wp.py

@@ -483,7 +483,7 @@ def wp(boxsize, pimax, nthreads, binfile, X, Y, Z,
     from future.utils import bytes_to_native_str
     from Corrfunc.utils import translate_isa_string_to_enum,\
         return_file_with_rbins, convert_to_native_endian,\
-        is_native_endian, sys_pipes, process_weights
+        sys_pipes, process_weights
 
     weights, _ = process_weights(weights, None, X, None, weight_type, autocorr=True)
 

Corrfunc/theory/xi.py

@@ -194,7 +194,7 @@ def xi(boxsize, nthreads, binfile, X, Y, Z,
     from future.utils import bytes_to_native_str
     from Corrfunc.utils import translate_isa_string_to_enum,\
         return_file_with_rbins, convert_to_native_endian,\
-        is_native_endian, sys_pipes, process_weights
+        sys_pipes, process_weights
 
     weights, _ = process_weights(weights, None, X, None, weight_type, autocorr=True)