- changed status to open
- removed comment
Reduce time spent in cycling timelevels if there is only one timelevel.
I noticed in a run with gridfunctions with only 1 timelevels that a significant amount of time was spent cycling timelevels. The attached patch makes the routine bypass a lot of code, if a grid variable only has 1 timelevel. This reduces the total time spent in cycling timelevels from 417 s to 0.8 s for a run of 32767 iterations.
Keyword: performance
Keyword: optimization
Comments (6)
-
reporter -
- removed comment
Do you know in what parts of the code all the time was spent? Maybe these parts should be optimized anyway, so that other simulations could benefit as well.
-
reporter - removed comment
No, I only got one timer number for the whole routine. I could look at it in more detail with vtune but the time spent in the routine per call is only 0.012 s, so it might be difficult to get sensible numbers.
-
- changed status to open
- removed comment
It seems simple enough. Please apply.
-
reporter - changed status to resolved
- removed comment
Committed
-
- edited description
- changed status to closed
- Log in to comment