Opened 17 years ago

Last modified 17 years ago

#154 closed defect

quantity.get_vertex_values is slow — at Version 3

Reported by: anonymous Owned by: ole
Priority: normal Milestone:
Component: Efficiency and optimisation Version:
Severity: major Keywords:
Cc:

Description (last modified by ole)

Profiling of okushiri (yieldstep=0.05, finaltime=2) gave the following runtimes:

store_vertex_values_uniquely(True): 138.3s

store_vertex_values_uniquely(False): 505.6s

No storing: 28.2s

Profiling revealed that quantity.get_vertex_values is the culprit.

I suggest rewriting some of it in C and perhaps always use mean as the reduction operator. We don't really need any other.

This problem is worst in simulations with frequents storing.

Change History (3)

comment:1 Changed 17 years ago by ole

  • Description modified (diff)

comment:2 Changed 17 years ago by ole

  • Description modified (diff)

comment:3 Changed 17 years ago by ole

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