#17309: SubHypergraphSearch
-------------------------------------+-------------------------------------
       Reporter:  ncohen             |        Owner:
           Type:  enhancement        |       Status:  positive_review
       Priority:  major              |    Milestone:  sage-6.4
      Component:  combinatorial      |   Resolution:
  designs                            |    Merged in:
       Keywords:                     |    Reviewers:  Dima Pasechnik
        Authors:  Nathann Cohen      |  Work issues:
Report Upstream:  N/A                |       Commit:
         Branch:  u/ncohen/17309     |  d83413ad45c40ed1b474f521a2bd0164f6c99bbf
   Dependencies:                     |     Stopgaps:
-------------------------------------+-------------------------------------
Changes (by dimpase):

 * status:  needs_review => positive_review
 * reviewer:   => Dima Pasechnik


Comment:

 Replying to [comment:15 ncohen]:
 > > The stuff should be tested on a 32-bit system, just to make sure there
 is no arch-specific bugs...
 >
 > Well, you can if you have one but I tried to be careful with respect to
 that and explicitly wrote `uint64_t` when I needed a 64-bits integer.
 Usually the 'int' variables that I use represent the vertex of a
 hypergraph, so I do not think that it will overflow anytime soon `:-P`

 OK, it works both on 64 and 32 bit (arando is still alive).
 LGTM.

--
Ticket URL: <http://trac.sagemath.org/ticket/17309#comment:21>
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