#18762: Create coercion between diagram algebras and the symmetric group algebra
-------------------------------------+-------------------------------------
       Reporter:  ghseeli            |        Owner:
           Type:  enhancement        |       Status:  needs_work
       Priority:  minor              |    Milestone:  sage-6.9
      Component:  algebra            |   Resolution:
       Keywords:  diagram algebra,   |    Merged in:
  partition algebra, days65          |    Reviewers:
        Authors:  George H.          |  Work issues:
  Seelinger                          |       Commit:
Report Upstream:  N/A                |  2adcfeffa5d87f137ee9010fb69087a2b76e98a7
         Branch:                     |     Stopgaps:
  
u/ghseeli/create_coercion_between_diagram_algebras_and_the_symmetric_group_algebra|
   Dependencies:  #18720             |
-------------------------------------+-------------------------------------

Comment (by ghseeli):

 Vincent,

 Thank you for your feedback. I had intended to change the status from
 needs_review to needs_work when Travis made his comments 3 weeks ago, so I
 am sorry you felt compelled to review something that was already in need
 of a lot of work. I understand that methods must be documented and I will
 do so accordingly before there is a positive review. However, first, I
 needed to establish what is going on here, (see the earlier comments about
 weird coercion behavior) and that is still somewhat (?) in the air. I do
 agree that my git comments are rather terse, but at the same time, as you
 note, most of them are rather small and thus looking at the diff should be
 more than enough to get an idea of what is happening. Furthermore, as
 someone who develops software semi-professionally, it is not unusual that
 the number of commits exceeds the number of lines changed. Most software
 developers would say that the number of lines affected has little
 correlation with content. Finally, a lot of these commits are due to the
 age of the ticket. New versions of the beta need to be merged into the
 branch so that the feature doesn't depart from the current beta. I am
 sorry that my development cycle has been so slow that this has been
 necessary. I hope to finish this ticket soon. Thank you for your feedback
 and I hope you will be more satisfied with the end result when it goes
 back up for review.

 Best,
 George

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