Opened 6 years ago

Last modified 6 years ago

#20074 closed enhancement

QQbar cleaning 2 — at Version 1

Reported by: vdelecroix Owned by:
Priority: major Milestone: sage-7.1
Component: number fields Keywords:
Cc: Merged in:
Authors: Vincent Delecroix Reviewers:
Report Upstream: N/A Work issues:
Branch: u/vdelecroix/20074 (Commits, GitHub, GitLab) Commit: 9264ff161e50900ce17ca62393d70b8abc4b4bb6
Dependencies: #19954 Stopgaps:

Status badges

Description (last modified by vdelecroix)

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

Change History (1)

comment:1 Changed 6 years ago by vdelecroix

  • Branch set to u/vdelecroix/20074
  • Commit set to 9264ff161e50900ce17ca62393d70b8abc4b4bb6
  • Description modified (diff)
  • Status changed from new to needs_review

Last 10 new commits:

​b35d6f6Trac 19954: fix doctests
​9f39b58Trac 19954: fix sage_input doctests
​7ab3ffbTrac 19954: move the 34-gon as a doctest
​ec2e9aaTrac 19954: merge 7.1.beta1
​d1788a0Trac 19954: fix doctests
​94629dbTrac 19954: doctest independent of execution order
​cb2592cTrac 19954: typo in documentation
​1385753Trac 19954: documentation
​c1f5c5fTrac 19954: remove useless "gaussian" functions
​9264ff1Trac 20074: qqbar cleaning
Note: See TracTickets for help on using tickets.