Skip to content

Add support for providing XCSF with a fixed seed#14

Merged
rpreen merged 1 commit intoxcsf-dev:masterfrom
dpaetzel:add-rng-seed
Oct 16, 2021
Merged

Add support for providing XCSF with a fixed seed#14
rpreen merged 1 commit intoxcsf-dev:masterfrom
dpaetzel:add-rng-seed

Conversation

@dpaetzel
Copy link
Copy Markdown
Collaborator

Hi!

There are two main reasons for this PR:

  • When submiting many XCSF runs to a cluster, the current implementation leads to most of them starting with the same random seed (due to the dependence on the current time in rand_init).
  • Fixing the random seed is necessary for repeatability.

Note that I haven't used C for quite some time now and thus may have made mistakes; however, for me, the code of this PR seems to work–so far. 🙂

@rpreen rpreen merged commit e55a66d into xcsf-dev:master Oct 16, 2021
@rpreen
Copy link
Copy Markdown
Member

rpreen commented Oct 16, 2021

Thanks for this. I can confirm it works here, too.

@dpaetzel dpaetzel deleted the add-rng-seed branch October 17, 2021 06:52
@dpaetzel
Copy link
Copy Markdown
Collaborator Author

Great, thanks for merging!

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