#3827 closed defect (fixed)
dasslJacobian flag is rename to jacobian
Reported by: | Christoph <buchner@…> | Owned by: | adeas31 |
---|---|---|---|
Priority: | high | Milestone: | |
Component: | OMEdit | Version: | |
Keywords: | Cc: | wbraun, adeas31 |
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 Changed 8 years ago by Christoph <buchner@…>
comment:2 Changed 8 years ago by adrpo
- Cc wbraun 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 Changed 8 years ago by adrpo
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 Changed 8 years ago by wbraun
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 Changed 8 years ago by wbraun
- Component changed from *unknown* to Run-time
- Summary changed from 64bit Windows nightly gives invalid option dasslJacobian to dasslJacobian flag is rename to jacobian
comment:6 Changed 8 years ago by adrpo
- Cc adeas31 added
Maybe would be good to have a small piece of code that checks for the old flag, give a warning and use the new 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 Changed 8 years ago by wbraun
- Component changed from Run-time to OMEdit
- Owner changed from somebody to adeas31
comment:8 Changed 8 years ago by wbraun
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:9 Changed 8 years ago by Christoph <buchner@…>
@adrpo yes, I have used default flags
comment:10 Changed 8 years ago by adeas31
- Milestone changed from Future to 1.10.0
- Resolution set to fixed
- Status changed from new to 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.