Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tool randomfield may use current time stamp as seed now #25

Open
wants to merge 2 commits into
base: patch_2020
Choose a base branch
from

Conversation

reetzelhaft
Copy link
Contributor

The tool randomfield creates random numbers based on a seed that can be specified as input. The default value is seed=1. Thus, if no seed is specified, the generated FlowField is always the same.

I changed the tool to use the current time in second as the seed value, if seed is initially set to 1 (the default). Now,

  • Set the seed to a number different from 1 to create always the same FlowField.
  • Do NOT set the seed to have "random" behaviour.

PS: The random number generator is called in the function randomReal(a,b) in mathdefs.h.

@codecov-commenter
Copy link

Codecov Report

Merging #25 into patch_2020 will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           patch_2020      #25   +/-   ##
===========================================
  Coverage       26.84%   26.84%           
===========================================
  Files              64       64           
  Lines           15730    15730           
  Branches         6815     6815           
===========================================
  Hits             4222     4222           
  Misses          11470    11470           
  Partials           38       38           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d35beb0...3c5125b. Read the comment docs.

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.

None yet

2 participants