#3697 closed defect (fixed)
Much slower front-end from 31eaa49 to 56626b0
| Reported by: | Adrian Pop | Owned by: | somebody |
|---|---|---|---|
| Priority: | blocker | Milestone: | 1.9.4 |
| Component: | Frontend | Version: | v1.9.4-dev-nightly |
| Keywords: | Cc: | Martin Sjölund, Henning Kiel, Per Östlund, Francesco Casella |
Description
It seems the front-end got 4 times *slower* from Feb 16 to Feb 19.
Tested with model: ScalableTestSuite.Electrical.TransmissionLine.ScaledExperiments.TransmissionLineEquations_N_10
omc 56626b0
real 0m9.462s
user 0m32.697s
sys 0m0.858s
omc 31eaa49
real 0m2.035s
user 0m4.400s
sys 0m0.670s
Even the parser seems two times slower:
omc 56626b0 Feb 19 Notification: Performance of loadModel(ScalableTestSuite): time 3.942/3.942 omc 31eaa49 Feb 16 Notification: Performance of loadModel(ScalableTestSuite): time 1.314/1.314
As far as I can tell only the changes by hkiel and sjoelund.se happened from Feb 16 to 19 to the front-end and parsing.
Change History (6)
comment:1 by , 9 years ago
| Priority: | critical → blocker |
|---|
comment:2 by , 9 years ago
The offending commit is this one: 3cd16cc1ae550915ab71c6c109768d7ad8e1f335/OMCompiler.
Very strange as it should have made things faster but it really not the case!
comment:3 by , 9 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
Revert changes to SCodeUtil.mo in 55cb64/OMCompiler as they slow things down in the front-end 4 times. This should be investigated why it happens!
Made it a blocker for the final release, the offending commit(s) should be rolled back or fixed before we ship 1.9.4.