#14234: Restructuring Diagram/Partition Algebras to match category structure
-------------------------------------------------+-------------------------
       Reporter:  ghseeli                        |         Owner:
           Type:  enhancement                    |  AlexGhitza
       Priority:  minor                          |        Status:
      Component:  algebra                        |  needs_work
       Keywords:  Partition algebra, diagram     |     Milestone:
  algebra, days38, days40                        |  sage-5.12
        Authors:  Stephen Doty, Aaron Lauve,     |    Resolution:
  George H. Seelinger                            |     Merged in:
Report Upstream:  N/A                            |     Reviewers:  Travis
         Branch:                                 |  Scrimshaw
       Stopgaps:                                 |   Work issues:
                                                 |  Dependencies:  #10630
-------------------------------------------------+-------------------------

Comment (by tscrim):

 There are two reasons why I left it as `0.5` instead of `1/2`, the first
 is that `floor` is being called and it seemed somewhat troublesome to
 covert it to an integer, and the second is this what the old partition
 algebras used (which is why I didn't look too hard to getting around the
 first problem). Also IMO you're abusing the input with your `==` example,
 so it's an unfair test.

 I think that `da.partition_diagrams` is okay since its more of an internal
 helper function, but I'll change the output to be a list in both cases to
 match the output of the other such functions.

 For the `0.5 +/- 0.5`, since there is no division being performed and they
 are specified floating points, their bit representations are the same and
 it's a power of two, so +/- are both okay (here). If it was say `0.1`
 added 10 times, then it's slightly more worrisome (but I think it's still
 okay...)

 In any case, I'll look more deeply into seeing if I can convert this to
 using rationals, make sure the 1/2's are correct (and fix it if it's not),
 and fix those typos (including my spelling of propagate `:P`).

 Thanks for your comments Darij.

 Best,[[BR]]
 Travis

--
Ticket URL: <http://trac.sagemath.org/ticket/14234#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/groups/opt_out.


Reply via email to