sagemath/sage

Make algebras doctests ready for random seeds

Closed this issue · 16 comments

kliem commented

This ticket makes

sage -t --long --random-seed=n src/sage/algebras/

pass for different values n than just 0.

Depends on #29962

Component: doctest framework

Author: Jonathan Kliem

Branch/Commit: 46c0147

Reviewer: Markus Wageringel

Issue created by migration from https://trac.sagemath.org/ticket/29968

kliem commented

Changed dependencies from #29662 to #29962

kliem commented

Branch: public/29968

kliem commented

Commit: 72d4116

kliem commented

New commits:

da1c6bestart from a "random" random seed for doctesting
b7b836dmake random seed reproducible
eedbe5edocument random_seed
998b1b9default random seed 0 for now
1d7b00edash instead of underscore for command line options
72d4116make algebras fuzz ready
kliem commented
comment:3

Currently this QuaternionAlgebra(-11,-1).maximal_order().random_element() gives and element of QuaternionAlgebra(-11,-1). This seems somewhat strange to me.


New commits:

95a2fefMerge branch 'public/29968' of git://trac.sagemath.org/sage into public/29968-reb
445beefbetter doctests
kliem commented

Changed branch from public/29968 to public/29968-reb

kliem commented

Changed commit from 72d4116 to 445beef

Changed commit from 445beef to 8038412

Branch pushed to git repo; I updated commit sha1. New commits:

8038412fix doctest
kliem commented

New commits:

b31e2d5Merge branch 'public/29962' of git://trac.sagemath.org/sage into public/29962-reb
2f30dd9small fixes
b62f781doctests do not start from a random seed by default yet
1d99129fix merge conflict
46c0147Merge branch 'public/29968-reb' of git://trac.sagemath.org/sage into public/29968-reb2
kliem commented

Changed branch from public/29968-reb to public/29968-reb2

kliem commented

Changed commit from 8038412 to 46c0147

comment:6

Ok.

Reviewer: Markus Wageringel

kliem commented
comment:7

Thank you.

Changed branch from public/29968-reb2 to 46c0147