Changes between Version 4 and Version 7 of Ticket #15311


Ignore:
Timestamp:
11/02/13 19:08:04 (6 years ago)
Author:
tscrim
Comment:

I was trying out new spellings of my names, and then copy/paste fun! XP

So I've folded in your review patch and made the following changes:

  • Fixed a bug so computations work for prime powers (I forgot to pass self._q to hall_polynomial()). This uncovered another problem, that [univariate] Laurent polynomials don't divide like the should (see #11726), so I changed all of the tests to use the fraction field.
  • Fixed the coercion.
  • Made HallAlgebra lazily imported.
  • Improved the doc and added some more doctests.

I didn't want to make this depend on #11726 since functionally it will work when Laurent polynomials do the division like they should (although it is horrendously slow with the current #11726 patch).

For patchbot:

Apply: trac_15311-hall_algebra-ts.patch

Legend:

Unmodified
Added
Removed
Modified