sagemath/sage

Make documentation doctests ready for random seeds

Closed this issue · 15 comments

kliem commented

This ticket makes

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

pass for different values n than just 0.

Depends on #29962

Component: doctest framework

Author: Jonathan Kliem

Branch/Commit: 0e062d5

Reviewer: Dima Pasechnik

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

kliem commented

Author: Jonathan Kliem

kliem commented

Branch: public/29981

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
f1066b4make documentation fuzz ready
kliem commented

Commit: f1066b4

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

90cb2f3Merge branch 'develop' of git://trac.sagemath.org/sage into public/29981

Changed commit from f1066b4 to 90cb2f3

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

ac45e07random event is valid now

Changed commit from 90cb2f3 to ac45e07

kliem commented
comment:4

Merge conflict.

I also need to go through it again and check I respected the design decision in #29935.

comment:6

Setting new milestone based on a cursory review of ticket status, priority, and last modification date.

Changed commit from ac45e07 to 0e062d5

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

0e062d5make documentation ready for random seeds

Reviewer: Dima Pasechnik

comment:9

lgtm

Changed branch from public/29981 to 0e062d5