Ticket #1201 (closed enhancement: fixed)

Opened 6 years ago

Last modified 5 years ago

[with patch, with positive review] add gramm-schmidt to sage

Reported by: was Owned by: was*
Priority: major Milestone: sage-2.8.15
Component: linear algebra Keywords:
Cc: Work issues:
Report Upstream: Reviewers:
Authors: Merged in:
Dependencies: Stopgaps:

Description

Add function to do gramm schmidt orthogonalization and also to verify the LLL criterion to Sage.

NOTE: I have mostly done this. This trac is just so I have a place to submit a patch. So you shouldn't do this :-).

Attachments

trac1201.patch Download (3.7 KB) - added by was 6 years ago.
7421.patch Download (822 bytes) - added by was 5 years ago.
also apply this patch (after the first)

Change History

Changed 6 years ago by was

comment:1 Changed 6 years ago by was

NOTE -- this is a generic implementation. There is surely a vastly faster implementation in the case of an RDF or CDF base field (by calling to numpy).

comment:2 Changed 6 years ago by was

From David Joyner: 

I looked at it but can't figure out how to apply it to a cloned version
of SAGE. It seems to make sense but I'd like to test it out.
Sorry for the delay. I had a final to type up and lots of grading.

comment:3 Changed 5 years ago by was

  • Milestone changed from sage-2.9 to sage-2.8.15

Changed 5 years ago by was

also apply this patch (after the first)

comment:4 Changed 5 years ago by mhansen

  • Summary changed from add gramm-schmidt to sage to [with patch] add gramm-schmidt to sage

comment:5 Changed 5 years ago by mhansen

  • Summary changed from [with patch] add gramm-schmidt to sage to [with patch, with positive review] add gramm-schmidt to sage

I think this can go in.

comment:6 Changed 5 years ago by mabshoff

  • Status changed from new to closed
  • Resolution set to fixed

Merged in 2.8.15.alpha2.

Note: See TracTickets for help on using tickets.