[ 
https://issues.apache.org/jira/browse/SANDBOX-404?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Claudio Squarcella updated SANDBOX-404:
---------------------------------------

    Attachment: SANDBOX-404_FromMonoidToAddition.patch

Hi Simone!

I completely agree with your last sentence, and that is one of the reasons why 
I am providing this extended patch where there is no {{Monoid}} anymore, only 
{{Addition}} -- simplification of the weight operations model! 

No use for theoretical algebra in the code: only clear, unambiguous names 
({{sum}}, {{negate}}, {{zero}}). I think that is a primary concern as long as 
we have the power to improve semantics before things go "live" (releases, 
people using the library consistently and getting used to class names, etc). 
Also, no risk to extend {{Monoid}} twice *if and when* we add 
{{Multiplication}} or whatsoever, because it will be totally independent. 

Hoping to meet your agreement and the one of others working on our beloved 
[graph] ;)
Claudio
                
> Simplify weight model
> ---------------------
>
>                 Key: SANDBOX-404
>                 URL: https://issues.apache.org/jira/browse/SANDBOX-404
>             Project: Commons Sandbox
>          Issue Type: Improvement
>          Components: Graph
>            Reporter: Simone Tripodi
>         Attachments: SANDBOX-404.patch, 
> SANDBOX-404_FromMonoidToAddition.patch, 
> SANDBOX-404_gettingRidOfOrderedMonoid.patch
>
>
> As discussed on {{dev@}}, {{Zero}}, {{Semigroup}} and {{Monoid}} can be 
> merged directly in one single interface

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to