Use le_layout for collisions by default

Merged
#3 · Created  · Last updated

Merged pull request

Merged in collisions_le_layout (pull request #3)

6d0e91f·Author: ·Closed by: ·2019-02-08

Description

Collisions can be calculated either in the le_layout, or in the two layouts e_layout and lz_layout. As I understand it, the two algorithms give identical results, but lz_layout/e_layout is much slower because of the addition memory redistributions/communications between the two layouts during the calculation.

However, it seems le_layout is not the default. Is there any reason why it shouldn’t be?

I also wonder if this should be a PR to master; the results are the same, so it’s a (performance) bugfix, not a breaking change.

0 attachments

0 comments

Loading commits...