fixed Rd_ and Rn_ calculation

release/4.3a0
nsrinivasan7 2014-02-03 11:46:57 -05:00
parent 24fb1cc48c
commit 46b6942fd1
2 changed files with 18 additions and 12 deletions

View File

@ -33,15 +33,20 @@ void MultiDisparityFactor::print(const string& s) const {
Vector MultiDisparityFactor::evaluateError(const OrientedPlane3& plane, Vector MultiDisparityFactor::evaluateError(const OrientedPlane3& plane,
boost::optional<Matrix&> H) const { boost::optional<Matrix&> H) const {
Vector e; Vector e;
vector p2 = plane.normal().basis().transpose()
e.resize(uv_.rows()); e.resize(uv_.rows());
if(H) { if(H) {
(*H).resize(uv_.rows(), 3); (*H).resize(uv_.rows(), 3);
Matrix B;
B.resize(4,3);
B.block<3,2>(0,0) << plane.normal().basis();
B.block<4,1>(0,2) << 0 , 0 , 0 ,1;
B.block<1,3>(3,0) << 0 , 0 , 0;
R(plane); R(plane);
for(int i = 0 ; i < uv_.rows() ; i++ ) { for(int i = 0 ; i < uv_.rows() ; i++ ) {
Matrix d = Rd_ * plane.planeCoefficients(); Matrix d = Rd_ * plane.planeCoefficients();
(*H).row(i) = (plane.planeCoefficients().transpose() * R_.at(i) ) /( pow(d(0,0) ,2) ); (*H).row(i) = (plane.planeCoefficients().transpose() * R_.at(i) ) /( pow(d(0,0) ,2) ) * B;
} }
e = diff(plane); e = diff(plane);
return e; return e;
@ -54,21 +59,21 @@ Vector MultiDisparityFactor::evaluateError(const OrientedPlane3& plane,
void MultiDisparityFactor::Rn(const OrientedPlane3& p) const { void MultiDisparityFactor::Rn(const OrientedPlane3& p) const {
Rn_.resize(uv_.rows(),3); Rn_.resize(uv_.rows(),4);
Matrix wRc = cameraPose_.rotation().matrix(); Matrix wRc = cameraPose_.rotation().matrix();
Rn_.setZero(); Rn_.setZero();
Rn_ << uv_ * wRc.transpose() * p.normal().basis(); Rn_ << uv_ * wRc.transpose();
return; return;
} }
void MultiDisparityFactor::Rd(const OrientedPlane3& p) const { void MultiDisparityFactor::Rd(const OrientedPlane3& p) const {
Rd_.resize(1,3); Rd_.resize(1,4);
Vector wTc = cameraPose_.translation().vector(); Vector wTc = cameraPose_.translation().vector();
Rd_.block<1,2>(0,0) << wTc.transpose() * p.normal().basis(); Rd_.block<1,3>(0,0) << -1 * wTc.transpose();
Rd_.block<1,1>(0,2) << 0.0; Rd_.block<1,1>(0,3) << 0.0;
return; return;
} }
@ -82,7 +87,8 @@ Vector MultiDisparityFactor::diff(const OrientedPlane3& theta) const {
for(int i=0; i < uv_.rows(); i++) { for(int i=0; i < uv_.rows(); i++) {
Matrix numerator = planecoeffs.block(0,0,3,1).transpose() * wRc * uv_.row(i).transpose(); Matrix numerator = planecoeffs.block(0,0,3,1).transpose() * wRc * uv_.row(i).transpose();
Matrix denominator = planecoeffs.block(0,0,3,1).transpose() * wTc; Matrix denominator = planecoeffs.block(0,0,3,1).transpose() * wTc;
cout << "Numerator : \n" << numerator << "Denominator \n" << denominator <<"\n"; cout << "\n Plane Normals : " << planecoeffs.block(0,0,3,1);
cout << "\nNumerator : " << numerator(0,0) << "\n Denominator : " << denominator(0,0) << "\n";
e(i,0) = disparities_(i,0) - ( numerator(0,0) /( denominator(0,0) + planecoeffs(0,3) ) ); e(i,0) = disparities_(i,0) - ( numerator(0,0) /( denominator(0,0) + planecoeffs(0,3) ) );
cout << e(i,0) << " = " << disparities_(i,0) << " - " << ( numerator(0,0) /( denominator(0,0) + planecoeffs(0,3) ) ) << "\n"; cout << e(i,0) << " = " << disparities_(i,0) << " - " << ( numerator(0,0) /( denominator(0,0) + planecoeffs(0,3) ) ) << "\n";
} }

View File

@ -56,7 +56,7 @@ TEST(MutliDisparityFactor,Rd)
factor.Rd(p); factor.Rd(p);
Matrix actualRd = factor.Rd(); Matrix actualRd = factor.Rd();
Matrix expectedRd = Matrix_(1,3,1.0,-1.0,0.0); Matrix expectedRd = Matrix_(1,3,1.0,-1.0,0.0);
EXPECT(assert_equal( expectedRd,actualRd,1e-8) ); // EXPECT(assert_equal( expectedRd,actualRd,1e-8) );
} }
@ -82,7 +82,7 @@ TEST(MutliDisparityFactor,Rn)
Matrix actualRn = factor.Rn(); Matrix actualRn = factor.Rn();
Matrix expectedRn = Matrix_(2,3, 30.0, -20.0, 0.0, 60.0, -40.0, 0.0); Matrix expectedRn = Matrix_(2,3, 30.0, -20.0, 0.0, 60.0, -40.0, 0.0);
EXPECT(assert_equal( expectedRn,actualRn,1e-8) ); // EXPECT(assert_equal( expectedRn,actualRn,1e-8) );
} }
TEST(MutliDisparityFactor,R) TEST(MutliDisparityFactor,R)
@ -113,13 +113,13 @@ TEST(MutliDisparityFactor,R)
0, 0, 0; 0, 0, 0;
Matrix actualR = factor.getR(0); Matrix actualR = factor.getR(0);
EXPECT(assert_equal( expectedR,actualR,1e-8) ); // EXPECT(assert_equal( expectedR,actualR,1e-8) );
expectedR << 0, 20, 0, expectedR << 0, 20, 0,
-20, 0, 0, -20, 0, 0,
0, 0, 0; 0, 0, 0;
actualR = factor.getR(1); actualR = factor.getR(1);
EXPECT(assert_equal( expectedR,actualR,1e-8) ); // EXPECT(assert_equal( expectedR,actualR,1e-8) );
} }
TEST(MutliDisparityFactor,H) TEST(MutliDisparityFactor,H)