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

Incorporate the latest bugfixes from the script #4

Open
7 of 9 tasks
mariusbarth opened this issue Apr 29, 2018 · 1 comment
Open
7 of 9 tasks

Incorporate the latest bugfixes from the script #4

mariusbarth opened this issue Apr 29, 2018 · 1 comment
Assignees

Comments

@mariusbarth
Copy link
Member

mariusbarth commented Apr 29, 2018

These are the commits that we need to check if they are already implemented in the package. Some of these are probably now redundant or need a little discussion. For example, I think warnings and messages should be included in the output object, not in a separate file. I also added "MB" for those commits that I will take a look at asap.

  • LOG_FILE option via sink(..)
  • new functions to check input: data, EQN_FILE (MB)
  • script should now work correctly for data sets with more than two levels (MB)
  • add seed argument to MPTinR script
  • write all meassages AND warnings to logfile
  • set random seed (only in script)
  • deactivate test settings (MB)
  • updated results for fixed random seed
  • PM data Smith et al., 2011 (MB)
@mariusbarth mariusbarth self-assigned this Jul 4, 2018
@danheck
Copy link
Collaborator

danheck commented Aug 1, 2018

set random seed (only in script) fixed in: 6914af9

note: by setting a random seed via the following option, it is not necessary to carry around an additional argument for the seed:
parallel::clusterSetRNGStream(cl, iseed = sample.int(.Machine$integer.max, 1))

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

No branches or pull requests

3 participants