#140: set_dynare_threads function fails on MATLAB < 7.2 --------------------------+------------------------------------------------- Reporter: sebastien | Owner: Type: bug | Status: new Priority: critical | Milestone: 4.2 Component: Core M-files | Version: Keywords: | --------------------------+------------------------------------------------- This is because setenv() was introduced in MATLAB 7.2.
We need to find a workaround for older versions.
Currently Dynare crashes on these versions for that reason.
#140: set_dynare_threads function fails on MATLAB < 7.2 ---------------------------+------------------------------------------------ Reporter: sebastien | Owner: stepan Type: bug | Status: assigned Priority: critical | Milestone: 4.2 Component: Core M-files | Version: Resolution: | Keywords: ---------------------------+------------------------------------------------ Changes (by sebastien):
* owner: => stepan * status: new => assigned
Comment:
Solution discussed by Stéphane and Sébastien:
- remove the environment variable - replace it by a substructure of options_, which would contain an entry per threaded DLL - add an extra mandatory input arg to every threaded DLL, corresponding to the number of threads: the argument will only be used if the DLL is compiled with OMP - add the extra input arg in every call from the MATLAB code, using the value from options_ - modify the set_dynare_threads function to add an extra input character arg, corresponding to the name of the DLL to affect; 'all' applies to all threaded DLL
#140: set_dynare_threads function fails on MATLAB < 7.2 ---------------------------+------------------------------------------------ Reporter: sebastien | Owner: stepan Type: bug | Status: closed Priority: critical | Milestone: 4.2 Component: Core M-files | Version: Resolution: fixed | Keywords: ---------------------------+------------------------------------------------ Changes (by stepan):
* status: assigned => closed * resolution: => fixed
Comment:
Resolved in commit #392872112a3669758b0817b36716aa09d219a072