#3827 closed defect (fixed)
dasslJacobian flag is rename to jacobian
Reported by: | Owned by: | Adeel Asghar | |
---|---|---|---|
Priority: | high | Milestone: | |
Component: | OMEdit | Version: | |
Keywords: | Cc: | Willi Braun, Adeel Asghar |
Description
I just downloaded the current Windows 64bit nightly, and when simulating examples, e.g. Modelica.Fluid.Examples.HeatingSystem, I get in the output tab:
stdout | OMEditInfo | C:/Users/buchner/AppData/Local/Temp/OpenModelica/OMEdit/Modelica.Fluid.Examples.HeatingSystem.exe -port=50112 -logFormat=xml -override=startTime=0,stopTime=6000,stepSize=12,tolerance=0.0001,solver=dassl,outputFormat=mat,variableFilter=.* -r=Modelica.Fluid.Examples.HeatingSystem_res.mat -dasslJacobian=coloredNumerical -w -lv=LOG_STATS
stdout | warning | invalid command line option: -dasslJacobian=coloredNumerical
changing the dass jacobian setting in simulation setup has no effect on this.
This is on OMEdit v1.10.0-dev-62-gc84ef06 connected to OpenModelica v1.10.0-dev-237-g583180a, 64bit on Windows 10. I uninstalled the previously installed 1.9.4 32bit before.
Change History (11)
comment:1 by , 9 years ago
comment:2 by , 9 years ago
Cc: | added |
---|
Hm. We only changed the compiler to gcc 5.3.0 from gcc 4.4.0, nothing more in the builds.
I will check if we still support the flag as there were some changes to the C runtime recently, maybe they are to blame.
@wbraun any comments on this?
comment:3 by , 9 years ago
Hi, it seems that 96898a5de5961eae09f5cfd178ff99d40cc915d0/OMCompiler removed that flag by generalization. @wbraun maybe we should have deprecated the flag for a while by changing it internally to the new ones.
comment:4 by , 9 years ago
Yes, I renamed the flag, since I want to use it also for other solvers then dassl.
It is now called just "-jacobian".
@adrpo Do you think we should add the other flag name and mark it as deprecated?
comment:5 by , 9 years ago
Component: | *unknown* → Run-time |
---|---|
Summary: | 64bit Windows nightly gives invalid option dasslJacobian → dasslJacobian flag is rename to jacobian |
comment:6 by , 9 years ago
Cc: | added |
---|
Maybe would be good to have a small piece of code that checks for the old flag, give a warning and use the old one. At least until people update their scripts.
I even wonder if we don't actually have this flag set by OMEdit as Christoph seems to have used the default flags.
@adeas31 do we give this flag in OMEdit by default?
comment:7 by , 9 years ago
Component: | Run-time → OMEdit |
---|---|
Owner: | changed from | to
comment:8 by , 9 years ago
Ah, it is OMEdit that use also dasslJacobian. I forgot about that. So actually we just need to adjust that in OMEdit.
@adeas31 Could you do that?
comment:10 by , 9 years ago
Milestone: | Future → 1.10.0 |
---|---|
Resolution: | → fixed |
Status: | new → closed |
Updated the flag used in OMEdit in 2d4e99c/OMEdit.
This does not occur with OMEdit v1.10.0-dev-53-g45d32fb, Connected to OpenModelica v1.10.0-dev-196-g8bddad5, which I just installed instead.