parcels icon indicating copy to clipboard operation
parcels copied to clipboard

Speedup C-contiguous arrays

Open erikvansebille opened this issue 4 years ago • 1 comments

This PR implements the 'engineering speedup' proposed by @CKehl in #957 by making sure data arrays are C-contiguous from the start (so that they don't need to be transposed on the fly)

erikvansebille avatar Aug 17 '21 11:08 erikvansebille

Sure, I understand and was not planning to merge this PR before you had completed your benchmarks. I was planning to use this branch to do some benchmarking of its speedup on of my own simulations.

Let's then merge this PR asap (surely before the next Parcels release) so that users can take advantage of it too; would be a shame to deny them this potential speed-up for their simulations!

erikvansebille avatar Sep 01 '21 05:09 erikvansebille