Skip to content
Snippets Groups Projects
Commit e6566215 authored by Johannes Pfeifer's avatar Johannes Pfeifer
Browse files

Eliminate set_dynare_seed.m calls with globals

parent f2abdb6e
Branches
Tags
1 merge request!2187Eliminate some more globals
Showing
with 148 additions and 106 deletions
Loading
  • This commit introduces two flavours of set_dynare_seed, one with globals and another one without. This is a bit confusing. And nothing guarantees that there won’t be any more use of the version with globals in the future.

    I’d rather transform set_dynare_seed into a command parsed by the preprocessor, which calls the .m version without globals (and we could thus get rid of the .m version with globals).

  • Johannes Pfeifer @JohannesPfeifer ·
    Author Developer

    If you could implement the preprocessor change, I would be more than willing to implement the desired Matlab change. Essentially, pass options_ as the first input and the only output.

  • Since this commit is a step towards that design, I’m going to merge it. Please open an issue regarding the improvement discussed above.

0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment