#20074: QQbar cleaning 2
-------------------------------------+-------------------------------------
       Reporter:  vdelecroix         |        Owner:
           Type:  enhancement        |       Status:  needs_review
       Priority:  major              |    Milestone:  sage-7.1
      Component:  number fields      |   Resolution:
       Keywords:                     |    Merged in:
        Authors:  Vincent Delecroix  |    Reviewers:
Report Upstream:  N/A                |  Work issues:
         Branch:                     |       Commit:
  u/vdelecroix/20074                 |  9264ff161e50900ce17ca62393d70b8abc4b4bb6
   Dependencies:  #19954             |     Stopgaps:
-------------------------------------+-------------------------------------
Changes (by vdelecroix):

 * status:  new => needs_review
 * commit:   => 9264ff161e50900ce17ca62393d70b8abc4b4bb6
 * branch:   => u/vdelecroix/20074


Old description:

> We further simplify QQbar code by:
> - using python operator to indentify binary operators instead of strings.
> In other words we replace `'+'` by `operator.add`, `'-'` by
> `operator.sub`, etc
> - writing only one function `binop` instead of `addsub` and `muldiv`.
> - removing the method `kind` of descriptors and instead use the class
> themselves for dispatching
> - detect unions earlier in the code to avoid constructing `ANBinaryExpr`

New description:

 We further simplify QQbar code by:
 - using python operator to indentify binary operators instead of strings.
 In other words we replace `'+'` by `operator.add`, `'-'` by
 `operator.sub`, etc
 - writing only one function `binop` instead of `addsub` and `muldiv`.
 - removing the method `kind` of descriptors and instead use the class
 themselves for dispatching
 - detect unions earlier in the code to avoid constructing `ANBinaryExpr`

 follow up: #19955

--

Comment:

 Last 10 new commits:
 
||[http://git.sagemath.org/sage.git/commit/?id=b35d6f613b69aec111336979bda0a7533eda31bf
 b35d6f6]||{{{Trac 19954: fix doctests}}}||
 
||[http://git.sagemath.org/sage.git/commit/?id=9f39b58ca6e7ac2689507b727d0a7afc9adec31f
 9f39b58]||{{{Trac 19954: fix sage_input doctests}}}||
 
||[http://git.sagemath.org/sage.git/commit/?id=7ab3ffbfd3e824c73a1c78d59de84b37f4b94e2c
 7ab3ffb]||{{{Trac 19954: move the 34-gon as a doctest}}}||
 
||[http://git.sagemath.org/sage.git/commit/?id=ec2e9aa7e43597a00407eca5358e5c74362d98fb
 ec2e9aa]||{{{Trac 19954: merge 7.1.beta1}}}||
 
||[http://git.sagemath.org/sage.git/commit/?id=d1788a00924a77f218bbff8c74d0eb56a7afa55c
 d1788a0]||{{{Trac 19954: fix doctests}}}||
 
||[http://git.sagemath.org/sage.git/commit/?id=94629dbef9c590f0f748d1c0e40e5833f226c245
 94629db]||{{{Trac 19954: doctest independent of execution order}}}||
 
||[http://git.sagemath.org/sage.git/commit/?id=cb2592cf73f815ccb611babbd18a3be19e95b235
 cb2592c]||{{{Trac 19954: typo in documentation}}}||
 
||[http://git.sagemath.org/sage.git/commit/?id=13857532e159d281181bb1b839a5c4f4ab937db7
 1385753]||{{{Trac 19954: documentation}}}||
 
||[http://git.sagemath.org/sage.git/commit/?id=c1f5c5fb96bf5bb06c4dacdaf132d7ce2b3c8de3
 c1f5c5f]||{{{Trac 19954: remove useless "gaussian" functions}}}||
 
||[http://git.sagemath.org/sage.git/commit/?id=9264ff161e50900ce17ca62393d70b8abc4b4bb6
 9264ff1]||{{{Trac 20074: qqbar cleaning}}}||

--
Ticket URL: <http://trac.sagemath.org/ticket/20074#comment:1>
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 https://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.

Reply via email to