#19340: Better interface for hadamard_matrix
-------------------------------------+-------------------------------------
       Reporter:  ncohen             |        Owner:
           Type:  enhancement        |       Status:  needs_review
       Priority:  major              |    Milestone:  sage-6.9
      Component:  combinatorial      |   Resolution:
  designs                            |    Merged in:
       Keywords:                     |    Reviewers:
        Authors:  Nathann Cohen      |  Work issues:
Report Upstream:  N/A                |       Commit:
         Branch:  u/ncohen/19340     |  247a2bb4dcec180227cd9fb9bcd331a27a289016
   Dependencies:                     |     Stopgaps:
-------------------------------------+-------------------------------------

Comment (by fredrik.johansson):

 You might consider using the flint functions fmpz_mat_hadamard and
 fmpz_mat_is_hadamard, which should be faster.

 Checking by default seems kind of useless since this is much slower than
 generating the matrix (generating a size 1000 Hadamard matrix takes 0.01
 seconds and checking it takes 0.5 seconds), and unit testing should
 provide reasonable certainty that the algorithm is correct. Flint's test
 suite already checks for every size up to 300 that fmpz_mat_hadamard
 either constructs a correct Hadamard matrix or reports that the size is
 unsupported.

--
Ticket URL: <http://trac.sagemath.org/ticket/19340#comment:6>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica, 
and MATLAB

-- 
You received this message because you are subscribed to the Google Groups 
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.

Reply via email to