Opened 4 years ago

Last modified 2 years ago

#18390 new enhancement

cleanup variable factories — at Version 1

Reported by: vdelecroix Owned by:
Priority: major Milestone: sage-6.7
Component: algebra Keywords:
Cc: mforets Merged in:
Authors: Vincent Delecroix Reviewers:
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description (last modified by vdelecroix)

The PolynomialRing factory is written in a very complicated way and is slightly different to FreeAlgebra. In this ticket, we normalize the inputs of all factories involving variables.

Some changes are noticeable

  1. the following will now raise TypeError
    sage: R.<a,b> = PolynomialRing(QQ, ('a', 'b'))
    sage: A.<x,y,z> = FreeAlgebra(QQ, 'x,y,z')
    
  1. The var_array argument in PolynomialRing is deprecated

As a sided effect, we got some speedup in polynomial ring creation!

Change History (1)

comment:1 Changed 4 years ago by vdelecroix

  • Description modified (diff)
  • Summary changed from cleanup polynomial ring factory to cleanup variable factories
Note: See TracTickets for help on using tickets.