Opened 8 years ago

Closed 8 years ago

Last modified 6 years ago

#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@…>

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.

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?

Last edited 8 years ago by adrpo (previous) (diff)

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.

comment:11 Changed 6 years ago by sjoelund.se

  • Milestone 1.10.0 deleted

Milestone deleted

Note: See TracTickets for help on using tickets.