Opened 7 years ago

Last modified 6 years ago

#4626 closed defect

Unit checking in the new front end is slow — at Version 1

Reported by: Francesco Casella Owned by: Mahder Alemseged Gebremedhin
Priority: blocker Milestone: 1.14.0
Component: OMEdit Version:
Keywords: Cc: Per Östlund, arunkumar palanisamy, Adeel Asghar

Description (last modified by Francesco Casella)

Even after the fixes of #4615, the unit-checking feature is the performance bottleneck of the new front-end, even for models that actually don't have unit information in their variables (see, e.g. the AdvectionReaction models of the ScalableTestSuite).

This may not be a big problem, because unit testing usually makes sense when developing new models, and that is normally done using small scale test models, so making this module faster is really not a high priority.

I would suggest that @perost changes the default behaviour to turn off unit checking by default on the new front end, so that we see the performance of stuff that reall matter, and then @mahge930 and @arun3688 have a closer look at performance issues and try to improve them, if this is not too hard.

@mahge930, please make sure there are test cases in the testsuite where unit checking is switched on after the default is changed.

Change History (1)

comment:1 by Francesco Casella, 7 years ago

Description: modified (diff)
Note: See TracTickets for help on using tickets.