Skip to content

Conversation

@jerebertho
Copy link
Contributor

@jerebertho jerebertho commented Dec 22, 2025

  1. This PR introduces the possibility for the user to give the seed to initialize the pseudo-random number generator with option -a (like alea). If 0 is given, then time(0) is used as before. Otherwise, the given positive number N is used.
    Observe that the latter should only be used for debug purpose only.
    It also cleans the code as only one call to srand is performed now.

  2. A bug was introduced in positive characteristic with No more random linear form over Q #249 where the introduction of a random linear form would make msolve quit at some point without outputting the result. This PR fixes this behaviour.

@jerebertho jerebertho marked this pull request as draft December 22, 2025 14:02
@jerebertho jerebertho marked this pull request as ready for review December 22, 2025 14:45
@vneiger
Copy link
Contributor

vneiger commented Dec 22, 2025

Thanks for the enhancements! Suggestions:

  • using uint32_t for the seed, instead of int32_t
  • maybe something like "-rs" (random seed) instead of "-a", if possible?

Co-authored-by: Vincent Neiger <vneiger@users.noreply.github.com>
@jerebertho jerebertho marked this pull request as draft December 22, 2025 19:59
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.

2 participants