Open martinmodrak opened 2 years ago
Hi @martinmodrak , Thank you for your feedback. I have the same issue. The model included a compartment with "default" id. This is a reserved word in mrgsolve, as I understand.
I've updated the repository with jak2stat5 model changing "default" => "def". Now everything is fine. Compilation and simulations are OK. Pull the latest commit from hetalang/qsp-benchmark and recompile the model.
This is a workaround. But the unexpected issues with reserved words are still a problem. We need a better solution.
Wouldn't a systematic solution be that the XXExport
classes expose a list of keywords and the compiler checks the model against this list. It could then either:
(I don't insist on this being done for the review, it just seems like a relatively cheap but useful solution)
@martinmodrak, agree, this would be good. As a fast temporal solution, I've added "default" into a general keyword list. Now the usage of such id throws an error. A user should change the id of such a component by himself. But I still doubt if it's the best solution. 1) It seems that each "export" type should have its own stop list. We need time to collect and analyze them all. 2) We should also think of automatic renaming rules.
So I guess we suggest a better solution later.
I took the model at https://github.com/hetalang/qsp-benchmark/tree/master/jak2stat5/src, compiled it and tried to run it with
mrgsolve
. Unfortunately, I get an error.I was able to run the simple example model from readme via mrgsolve, so it presumably isn't just an installation issue.
The error I get is: