Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make Gram-Schmidt work again #12646

Open
sagetrac-johanbosman mannequin opened this issue Mar 9, 2012 · 2 comments
Open

Make Gram-Schmidt work again #12646

sagetrac-johanbosman mannequin opened this issue Mar 9, 2012 · 2 comments

Comments

@sagetrac-johanbosman
Copy link
Mannequin

sagetrac-johanbosman mannequin commented Mar 9, 2012

sage: M = Matrix(RealField(100), 2, 2, [1, 2, 3, 4])
sage: M.gram_schmidt()
---------------------------------------------------------------------------
NotImplementedError                       Traceback (most recent call last)
...
NotImplementedError: Gram-Scmidt orthogonalization not implemented for matrices over inexact rings, except for RDF and CDF

This used to work in older Sage versions. And Schmidt lacks an h in the error message.

CC: @rbeezer

Component: linear algebra

Keywords: Gram-Schmidt

Issue created by migration from https://trac.sagemath.org/ticket/12646

@koffie
Copy link

koffie commented Apr 26, 2012

comment:1

I ran in to this also when trying to use Johan his code at http://www.sagenb.org/home/pub/3154/ . It seems that this was changed in #10791 . I added Rob Beezer as a CC since he might probably have something usefull to say on how to fix this (I'm not blaming you Rob, reading the ticket the general implementation was broken and the old code is still available in modules.misc).

ps. Johan: you might find the following code snippet usefull:

#this broke because of #10791
#G, _ = M.gram_schmidt()
#this is a workaround:
from sage.modules.misc import gram_schmidt
G, _ = gram_schmidt(list(M))
v = []

@rbeezer
Copy link
Mannequin

rbeezer mannequin commented Apr 27, 2012

comment:2

Replying to @koffie:

I added Rob Beezer as a CC since he might probably have something useful to say

Thanks, Marten, for including me.

  1. I wouldn't trust the old code on RealField(100) (or much else for that matter).
  2. The QR decomposition for RDF/CDF matrices would be my alternate suggestion. That's only 53-bit so that could be a downside if you need more precision.
  3. Or as Marten has suggested, all the old code is still available and unharmed if you want to take the extra step to import it.

Rob

@jdemeyer jdemeyer modified the milestones: sage-5.11, sage-5.12 Aug 13, 2013
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.1, sage-6.2 Jan 30, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.2, sage-6.3 May 6, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.3, sage-6.4 Aug 10, 2014
@mkoeppe mkoeppe removed this from the sage-6.4 milestone Dec 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants