Loading Minsky model file throws constant deprecated error
System dynamics program with additional features for economics
Brought to you by:
hpcoder,
profstevekeen
Diff:
I think this is the file
Steve does actually mention you should run the bleeding edge Minsky beta in his blog post. I can verify that 2.18 crashes on loading this file, but that 2.19-beta.14 correctly loads and runs it.
Simplest thing for you Hedley is to "apt get minsky-beta". But longer term you'll need to decide whether you want to run the latest release, or the latest bleeding edge beta, the latter very likely being less stable.
In the meantime, we do need to investigate why 2.18 crashes with this input.
2 things:
1) There is bogus TensorInit data in the .mky file. Will need to investigate whether there is an error in minsky-beta causing this.
2) Classdesc container unpack routines should check that there is sufficient data in the pack_t buffer, and throw before attempting to allocate the container. This should prevent the out of memory conditions of the crash.
Bit more work here than I realised, but the basic philosophy carried through of avoiding size_t as a data type. It appears Windows was 32 bits, and Linux 64 bits. I now have Windows and Linux software shipping .mky files between each other.
What this does mean is that .mky files with tensor values parameters created by the beta version of Minsky on Windows will not be valid, and will need their data reloaded from csv files.