Skip to content

Conversation

@frankluebeck
Copy link
Member

  • removed unnecessary index entry "random element of .."
  • added a new index entry in the general documentation of
    Random which only consists of the text "Random" and so
    will be the first match when a user types ?Random
  • in last mentioned section: changed wording about PseudoRandom
    and improved examples
  • corrected labels and added an example in ManSection on
    Random with a random source argument

This addresses issue " Merge documentation entries for 'Random'" #116

- removed unnecessary index entry "random element of .."
- added a new index entry in the general documentation of
  `Random` which only consists of the text "Random" and so
  will be the first match when a user types `?Random`
- in last mentioned section: changed wording about `PseudoRandom`
  and improved examples
- corrected labels and added an example in ManSection on
  `Random` with a random source argument
stevelinton added a commit that referenced this pull request Dec 18, 2015
@stevelinton stevelinton merged commit ace95e5 into gap-system:master Dec 18, 2015
@olexandr-konovalov
Copy link
Member

The diffs below occur in the test of manual examples. I will fix them.

########> Diff in [ "./../../lib/random.gd", 63, 67 ]
# Input is:
Random(mysource, 1, 10^60);
# Expected output:
474943581767832038339909502952186501785153672757525589711853
# But found:
999331861769949319194941485000557997842686717712198687315183
########
########> Diff in [ "./../../lib/coll.gd", 1621, 1631 ]
# Input is:
Random([1..6]);
# Expected output:
6
# But found:
1
########
########> Diff in [ "./../../lib/coll.gd", 1621, 1631 ]
# Input is:
Random(1, 2^100);
# Expected output:
216763054510766614871671341272
# But found:
866227015645295902682304086250
########
########> Diff in [ "./../../lib/coll.gd", 1621, 1631 ]
# Input is:
g:= Group( (1,2,3) );;  Random( g );  Random( g );
# Expected output:
()
(1,3,2)
# But found:
(1,3,2)
(1,2,3)
########
########> Diff in [ "./../../lib/coll.gd", 1621, 1631 ]
# Input is:
Random(Rationals);
# Expected output:
-4
# But found:
-2
########

olexandr-konovalov pushed a commit that referenced this pull request Dec 20, 2015
hulpke pushed a commit to hulpke/gap that referenced this pull request Dec 21, 2015
hulpke added a commit to hulpke/gap that referenced this pull request Dec 21, 2015
* commit '2ee33a75ae033755b6fce81551bf7c5ffc1594e1':
  Updated index entries for arg and Argument
  Added /:=, Fix to List(Filtered construct
  Make `Filtered' and `List(Filtered' its own parsed unit
  Parse `Filtered' construct.
  Translate ` as record component. <...> as list. Further function names.
  Declare second argument as IsObject as IsInt is not yet available
  Updated examples changed after PR gap-system#410
  Fixed examples after PR gap-system#391
  improve auto generated manual example test files, add some auto-generated files to .gitignore
  Properly translate `elif' even if parse tree uses `else if'
  Facilty to translate function names to GAP equivalents
  Remove extern/gmp-5.0.5.tar.gz
  Better match list for `?Random`
  Remove arg representing variadic arguments in function lists, except when it is the only argument
  Add ... style varargs with tests and documentation
  Build GMP with C++ support enabled
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants