#3697 closed defect (fixed)
Much slower front-end from 31eaa49 to 56626b0
| Reported by: | adrpo | Owned by: | somebody |
|---|---|---|---|
| Priority: | blocker | Milestone: | 1.9.4 |
| Component: | Frontend | Version: | v1.9.4-dev-nightly |
| Keywords: | Cc: | sjoelund.se, hkiel, perost, 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 Changed 9 years ago by casella
- Priority changed from critical to blocker
comment:2 Changed 9 years ago by adrpo
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 Changed 9 years ago by adrpo
- Resolution set to fixed
- Status changed from new to 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!
comment:4 Changed 9 years ago by adrpo
Closed this one and opened #3699 to deal with the underlying cause.
comment:5 Changed 8 years ago by sjoelund.se
- Milestone changed from 1.9.4 to 1.9.4-1.9.x
Milestone renamed
comment:6 Changed 8 years ago by sjoelund.se
- Milestone changed from 1.9.4-1.9.x to 1.9.4
Milestone renamed
Made it a blocker for the final release, the offending commit(s) should be rolled back or fixed before we ship 1.9.4.