On 21 Oct 2011, at 13:13, Carnë Draug wrote: > 2011/10/20 c. <carlo.defa...@gmail.com>: >> On 20 Oct 2011, at 17:13, Carnë Draug wrote: >>> I noticed that you have implemented 2 versions of the pgmres function >>> in the linear-algebra package, one in C++ and another in octave. >>> Should we remove one of them? If so, which? >> >> the m-file version was included in Octave-core as "gmres.m" so it is OK to >> remove it from linear-algebra. > > Removed. > >> the C++ version has the same functionality but is (very slightly) faster so >> you might want to keep it. > > If it has the same functionality but works faster, why not use it in > octave-core too? > Is there a reason for that other than the slightly > different API?
If I recall correctly the main reason to switch to m-file was because the m-file version can handle complex matrices while the C++ would need to be adapted, the speed difference was found to not be worth the loss of generality. > >> Also mgorth.cc which is used by gmres is now in Octave-core so is not needed >> there. > > Removed > >> if you are cleaning up linear algebra you might also want to look at other >> functions that are now in >> /scripts/sparse/ in the Octave-core source tree (*) in order to remove >> duplicates. > > Also removed sprand and bicg. Thank you for pointing this out. > > Carnë c. ------------------------------------------------------------------------------ The demand for IT networking professionals continues to grow, and the demand for specialized networking skills is growing even more rapidly. Take a complimentary Learning@Cisco Self-Assessment and learn about Cisco certifications, training, and career opportunities. http://p.sf.net/sfu/cisco-dev2dev _______________________________________________ Octave-dev mailing list Octave-dev@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/octave-dev
