Opened 18 years ago

Last modified 18 years ago

#154 closed defect

quantity.get_vertex_values is slow — at Version 2

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 (2)

comment:1 Changed 18 years ago by ole

Description: modified (diff)

comment:2 Changed 18 years ago by ole

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