#11379: Add Quantumino solver to sage/games
---------------------------+------------------------------------------------
   Reporter:  slabbe       |          Owner:  slabbe    
       Type:  enhancement  |         Status:  needs_work
   Priority:  major        |      Milestone:  sage-4.7.1
  Component:  misc         |       Keywords:  sd31      
Work_issues:               |       Upstream:  N/A       
   Reviewer:  Rob Beezer   |         Author:            
     Merged:               |   Dependencies:            
---------------------------+------------------------------------------------
Changes (by slabbe):

  * status:  needs_review => needs_work


Comment:

 Hi Rob,

 > Sorry to be so tardy on this one.  Really just one "issue" that I think
 needs attention.

 No problem!

 > Made a reviewer patch:  Changed some module and class links in
 documentation to be active, fixed a couple minor English language things.
 Do '''not''' list me as an author for these.

 Good, thanks for those fixes! Is there a problem with one of the fixes ?
 Because there is a symbol tilde {{{~}}} that appears in front of one of
 the class path.

 {{{
 :class:`~sage.combinat.tiling.TilingSolver`
 }}}

 > The animations are great.  Can you do something to mark the end (like a
 few blank frames, for maybe a half-second)?  It goes so fast, it is hard
 to tell where the start is and where the end is.

 The delay between frames and the number of iterations are arguments of the
 method show (see {{{animate??}}}). I would keep the animation function as
 is, but add an exemple in the doctest of how to change those parameters
 and add blank frames at the end. What do you think?

 >     1.  Pentamino 8 (a yellow one) still has a hole in it.  Not a big
 deal, but perhaps a symptom of something that should be done more
 carefully?

 Ok. I will think about it. Let me find a solution which will be better
 than the "cube in the middle" I am using up to now. Maybe using Simplicial
 Complexes of cubes, I could get the exact boundary of the piece? I take a
 look at it and comes back with a fix soon.

 Sébastien

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11379#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 post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/sage-trac?hl=en.

Reply via email to