Opened 16 years ago
Closed 12 years ago
#1085 closed defect (fixed)
Speedup check of Digital examples (from MathCore)
Reported by: | krsta | Owned by: | krsta |
---|---|---|---|
Priority: | normal | Milestone: | 1.9.0 |
Component: | Frontend | Version: | |
Keywords: | Cc: | krsta |
Description
Running checkmodel on digital examples gives the follwing times (in seconds):
timing(checkModel(Modelica.Electrical.Digital.Examples.Adder4))
126.219
timing(checkModel(Modelica.Electrical.Digital.Examples.Counter))
423.706
Implementation note:
The bottleneck is the large array constants defined in the Digital libraries, which are evaluated during the lookup process. The solution is probably to skip evaluation of bindings during partial instantiation, but the effects of this must be investigated.
Change History (8)
comment:1 by , 15 years ago
comment:2 by , 14 years ago
This is now down to 7.32/12.69 seconds for me. Is that acceptable for now?
comment:4 by , 14 years ago
And now 2.7 + 3.73 (all without weird things like not elaborating enums)
comment:5 by , 14 years ago
From brugard via email: Looking much better now. The question though is if we should close it or not. It is still about a factor 3-5 slower than Dymola, so there is still some work to do. My suggestion would be to decrease priority to normal.
comment:6 by , 13 years ago
On my machine now down to 1.73+1.04. 1.55+0.95 on the test-server. I did not do any changes; these are from general speed-up of the compiler (possibly related to changing the OMC counting of simple equations).
comment:7 by , 12 years ago
Cc: | krsta, → krsta |
---|---|
Component: | → Backend |
Milestone: | → Future |
Priority: | high → normal |
comment:8 by , 12 years ago
Component: | Backend → Frontend |
---|---|
Milestone: | Future → 1.9.0 |
Resolution: | → fixed |
Status: | new → closed |
Agreement during skype meeting this is working good enough for now
http://intranet/trac/mathmodelica/ticket/1551
This has been reduced with approximately a factor of three, however it is still way to long (Dymola less then 1 second).