Merge branch 'master' into improve_coverage

.travis.yml

@@ -7,9 +7,13 @@ python:
       - "3.6"
       - "3.6-dev"
       - "nightly" # currently points to 3.7-dev
+
+matrix:
+  allow_failures:
+    - python: "nightly"
+
 install:
     - "pip install --editable .[dev]"
-    - "pip install -r requirements-test.txt"
     - "pip install python-coveralls"
 script:
     - "pytest -v  --cov-report= --cov=quaternions tests/"

quaternions/general_quaternion.py

@@ -102,7 +102,10 @@ def normalized(self):
         return self / self.norm()
 
     def euclidean_distance(self, other):
-        """ Returns the euclidean distance between two quaternions. Note: differs from unitary quaternions distance. """
+        """
+        Returns the euclidean distance between two quaternions.
+        Note: differs from unitary quaternions distance.
+        """
         return (self - other).norm()
 
     def is_unitary(self, tolerance=DEFAULT_TOLERANCE):

quaternions/quaternion.py

@@ -4,7 +4,9 @@
 import numbers
 
 from quaternions.utils import (covariance_matrix_from_angles, sigma_lerner, xi_matrix)
-from quaternions.general_quaternion import GeneralQuaternion, QuaternionError, DEFAULT_TOLERANCE, exp
+from quaternions.general_quaternion import (
+    GeneralQuaternion, QuaternionError, DEFAULT_TOLERANCE, exp
+)
 
 import warnings
 
@@ -44,7 +46,9 @@ def __call__(self, p):
         try:
             return self * p
         except Exception as e:
-            raise QuaternionError('expected list of 3 elements, got %s, caused error: %s' % (p.__class__.__name__, e))
+            raise QuaternionError(
+                'expected list of 3 elements, got %s, caused error: %s' %
+                (p.__class__.__name__, e))
 
     def is_equal(self, other, tolerance=DEFAULT_TOLERANCE):
         """ compares as quaternions, i.e. up to sign. """
@@ -230,7 +234,7 @@ def from_qmethod(source, target, probabilities=None):
         K[1:4, 0] = [i, j, k]
         K[1:4, 1:4] = S - sigma * np.identity(3)
         return Quaternion._first_eigenvector(K)
-    
+
     @staticmethod
     def average_and_std_naive(*quaternions, weights=None):
         """

setup.py

@@ -31,8 +31,11 @@
         'numpy',
     ],
     extras_require={
-        "dev": [
-            "hypothesis",
+        'dev': [
+            'flake8 >= 2.5.4',
+            'hypothesis',
+            'pytest',
+            'pytest-coverage',
         ]
     }
 )

tests/test_quaternion.py

@@ -1,6 +1,6 @@
 import unittest
 
-from hypothesis import given, assume, strategies
+from hypothesis import given, assume, strategies, example
 import numpy as np
 import pytest
 import os
@@ -137,6 +137,7 @@ def test_basis(self, arr):
         assert np.array_equal([b1, b2, b3], q.matrix)
 
     @given(ANY_ROTATION_VECTOR)
+    @example([3.7500000000000733, 0.0, 3.5030739510277804e-05])
     def test_from_ra_dec_roll(self, arr):
         xyz = np.deg2rad(arr)
         c3, c2, c1 = np.cos(xyz)
@@ -147,7 +148,7 @@ def test_from_ra_dec_roll(self, arr):
             [s1 * s3 - c1 * c3 * s2, c3 * s1 + c1 * s2 * s3,  c1 * c2]   # noqa
         ]))
 
-        assert Quaternion.from_ra_dec_roll(*arr) == Quaternion.from_matrix(expected)
+        assert Quaternion.from_ra_dec_roll(*arr).is_equal(Quaternion.from_matrix(expected), tolerance=5e-8)
 
     @given(ANY_QUATERNION)
     def test_ra_dec_roll(self, arr):