X-Git-Url: https://www.ginac.de/ginac.git//ginac.git?a=blobdiff_plain;f=ginac%2Fmatrix.cpp;h=8a68b663b90293a318e445bf8f5801980490e901;hb=f7a07ae7330a3543bb00aa7b659e2624d6c440c7;hp=380a45e718b047a9c44db0bf2aa980813d03d012;hpb=9c91319253425e66f9b6feb9e7036f841bc55e84;p=ginac.git

diff --git a/ginac/matrix.cpp b/ginac/matrix.cpp
index 380a45e7..8a68b663 100644
--- a/ginac/matrix.cpp
+++ b/ginac/matrix.cpp
@@ -25,16 +25,17 @@
 #include <stdexcept>
 
 #include "matrix.h"
-#include "archive.h"
 #include "numeric.h"
 #include "lst.h"
 #include "idx.h"
 #include "indexed.h"
-#include "utils.h"
-#include "debugmsg.h"
 #include "power.h"
 #include "symbol.h"
 #include "normal.h"
+#include "print.h"
+#include "archive.h"
+#include "utils.h"
+#include "debugmsg.h"
 
 namespace GiNaC {
 
@@ -145,36 +146,35 @@ DEFAULT_UNARCHIVE(matrix)
 
 // public
 
-void matrix::print(std::ostream & os, unsigned upper_precedence) const
+void matrix::print(const print_context & c, unsigned level) const
 {
-	debugmsg("matrix print",LOGLEVEL_PRINT);
-	os << "[[ ";
-	for (unsigned r=0; r<row-1; ++r) {
-		os << "[[";
-		for (unsigned c=0; c<col-1; ++c)
-			os << m[r*col+c] << ",";
-		os << m[col*(r+1)-1] << "]], ";
-	}
-	os << "[[";
-	for (unsigned c=0; c<col-1; ++c)
-		os << m[(row-1)*col+c] << ",";
-	os << m[row*col-1] << "]] ]]";
-}
+	debugmsg("matrix print", LOGLEVEL_PRINT);
+
+	if (is_of_type(c, print_tree)) {
+
+		inherited::print(c, level);
+
+	} else {
+
+		c.s << "[[ ";
+		for (unsigned y=0; y<row-1; ++y) {
+			c.s << "[[";
+			for (unsigned x=0; x<col-1; ++x) {
+				m[y*col+x].print(c);
+				c.s << ",";
+			}
+			m[col*(y+1)-1].print(c);
+			c.s << "]], ";
+		}
+		c.s << "[[";
+		for (unsigned x=0; x<col-1; ++x) {
+			m[(row-1)*col+x].print(c);
+			c.s << ",";
+		}
+		m[row*col-1].print(c);
+		c.s << "]] ]]";
 
-void matrix::printraw(std::ostream & os) const
-{
-	debugmsg("matrix printraw",LOGLEVEL_PRINT);
-	os << class_name() << "(" << row << "," << col <<",";
-	for (unsigned r=0; r<row-1; ++r) {
-		os << "(";
-		for (unsigned c=0; c<col-1; ++c)
-			os << m[r*col+c] << ",";
-		os << m[col*(r-1)-1] << "),";
 	}
-	os << "(";
-	for (unsigned c=0; c<col-1; ++c)
-		os << m[(row-1)*col+c] << ",";
-	os << m[row*col-1] << "))";
 }
 
 /** nops is defined to be rows x columns. */
@@ -875,50 +875,32 @@ matrix matrix::inverse(void) const
 	if (row != col)
 		throw (std::logic_error("matrix::inverse(): matrix not square"));
 	
-	// NOTE: the Gauss-Jordan elimination used here can in principle be
-	// replaced by two clever calls to gauss_elimination() and some to
-	// transpose().  Wouldn't be more efficient (maybe less?), just more
-	// orthogonal.
-	matrix tmp(row,col);
-	// set tmp to the unit matrix
-	for (unsigned i=0; i<col; ++i)
-		tmp.m[i*col+i] = _ex1();
+	// This routine actually doesn't do anything fancy at all.  We compute the
+	// inverse of the matrix A by solving the system A * A^{-1} == Id.
 	
-	// create a copy of this matrix
-	matrix cpy(*this);
-	for (unsigned r1=0; r1<row; ++r1) {
-		int indx = cpy.pivot(r1, r1);
-		if (indx == -1) {
+	// First populate the identity matrix supposed to become the right hand side.
+	matrix identity(row,col);
+	for (unsigned i=0; i<row; ++i)
+		identity.set(i,i,_ex1());
+	
+	// Populate a dummy matrix of variables, just because of compatibility with
+	// matrix::solve() which wants this (for compatibility with under-determined
+	// systems of equations).
+	matrix vars(row,col);
+	for (unsigned r=0; r<row; ++r)
+		for (unsigned c=0; c<col; ++c)
+			vars.set(r,c,symbol());
+	
+	matrix sol(row,col);
+	try {
+		sol = this->solve(vars,identity);
+	} catch (const std::runtime_error & e) {
+	    if (e.what()==std::string("matrix::solve(): inconsistent linear system"))
 			throw (std::runtime_error("matrix::inverse(): singular matrix"));
-		}
-		if (indx != 0) {  // swap rows r and indx of matrix tmp
-			for (unsigned i=0; i<col; ++i)
-				tmp.m[r1*col+i].swap(tmp.m[indx*col+i]);
-		}
-		ex a1 = cpy.m[r1*col+r1];
-		for (unsigned c=0; c<col; ++c) {
-			cpy.m[r1*col+c] /= a1;
-			tmp.m[r1*col+c] /= a1;
-		}
-		for (unsigned r2=0; r2<row; ++r2) {
-			if (r2 != r1) {
-				if (!cpy.m[r2*col+r1].is_zero()) {
-					ex a2 = cpy.m[r2*col+r1];
-					// yes, there is something to do in this column
-					for (unsigned c=0; c<col; ++c) {
-						cpy.m[r2*col+c] -= a2 * cpy.m[r1*col+c];
-						if (!cpy.m[r2*col+c].info(info_flags::numeric))
-							cpy.m[r2*col+c] = cpy.m[r2*col+c].normal();
-						tmp.m[r2*col+c] -= a2 * tmp.m[r1*col+c];
-						if (!tmp.m[r2*col+c].info(info_flags::numeric))
-							tmp.m[r2*col+c] = tmp.m[r2*col+c].normal();
-					}
-				}
-			}
-		}
+		else
+			throw;
 	}
-	
-	return tmp;
+	return sol;
 }
 
 
@@ -981,8 +963,10 @@ matrix matrix::solve(const matrix & vars,
 	switch(algo) {
 		case solve_algo::gauss:
 			aug.gauss_elimination();
+			break;
 		case solve_algo::divfree:
 			aug.division_free_elimination();
+			break;
 		case solve_algo::bareiss:
 		default:
 			aug.fraction_free_elimination();