Merged in feature/testSphereSolution (pull request #6)
Made Sphere2 constructor to be explicit + some changes in Rot3.release/4.3a0
commit
c04eff465d
|
@ -73,7 +73,7 @@ Point3 Rot3::operator*(const Point3& p) const {
|
||||||
/* ************************************************************************* */
|
/* ************************************************************************* */
|
||||||
Sphere2 Rot3::rotate(const Sphere2& p,
|
Sphere2 Rot3::rotate(const Sphere2& p,
|
||||||
boost::optional<Matrix&> HR, boost::optional<Matrix&> Hp) const {
|
boost::optional<Matrix&> HR, boost::optional<Matrix&> Hp) const {
|
||||||
Sphere2 q = rotate(p.point3(Hp));
|
Sphere2 q = Sphere2(rotate(p.point3(Hp)));
|
||||||
if (Hp)
|
if (Hp)
|
||||||
(*Hp) = q.basis().transpose() * matrix() * (*Hp);
|
(*Hp) = q.basis().transpose() * matrix() * (*Hp);
|
||||||
if (HR)
|
if (HR)
|
||||||
|
@ -84,7 +84,7 @@ Sphere2 Rot3::rotate(const Sphere2& p,
|
||||||
/* ************************************************************************* */
|
/* ************************************************************************* */
|
||||||
Sphere2 Rot3::unrotate(const Sphere2& p,
|
Sphere2 Rot3::unrotate(const Sphere2& p,
|
||||||
boost::optional<Matrix&> HR, boost::optional<Matrix&> Hp) const {
|
boost::optional<Matrix&> HR, boost::optional<Matrix&> Hp) const {
|
||||||
Sphere2 q = unrotate(p.point3(Hp));
|
Sphere2 q = Sphere2(unrotate(p.point3(Hp)));
|
||||||
if (Hp)
|
if (Hp)
|
||||||
(*Hp) = q.basis().transpose() * matrix().transpose () * (*Hp);
|
(*Hp) = q.basis().transpose() * matrix().transpose () * (*Hp);
|
||||||
if (HR)
|
if (HR)
|
||||||
|
|
|
@ -60,7 +60,7 @@ public:
|
||||||
}
|
}
|
||||||
|
|
||||||
/// Construct from point
|
/// Construct from point
|
||||||
Sphere2(const Point3& p) :
|
explicit Sphere2(const Point3& p) :
|
||||||
p_(p / p.norm()) {
|
p_(p / p.norm()) {
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue