Uses array for hex grid property layer fix

[Sahil-Chhoker]

Summary

lru_cache does not work with generators, therefore _get_hexmesh is changed to a function.

This was a problem in #2646 that used generators which don't go well along with caching. This PR simply removes the generator and adds a list where all the results are stored.

[Sahil-Chhoker]

@quaquel I think this should do it.

[github-actions]

Performance benchmarks:

Model	Size	Init time [95% CI]	Run time [95% CI]
BoltzmannWealth	small	🔵 +1.8% [+1.2%, +2.4%]	🔵 +1.9% [+1.7%, +2.0%]
BoltzmannWealth	large	🔵 +0.2% [-1.1%, +0.9%]	🔵 +1.3% [+0.1%, +2.4%]
Schelling	small	🔵 +1.1% [+0.8%, +1.4%]	🔵 +0.7% [+0.5%, +0.8%]
Schelling	large	🔵 +0.8% [+0.4%, +1.1%]	🔵 -1.6% [-3.6%, +0.5%]
WolfSheep	small	🔵 -0.6% [-1.0%, -0.2%]	🔵 +0.2% [-0.0%, +0.3%]
WolfSheep	large	🔵 -2.3% [-4.8%, +0.0%]	🔵 -7.8% [-12.9%, -2.7%]
BoidFlockers	small	🔵 -1.9% [-2.9%, -0.9%]	🔵 +0.6% [+0.2%, +1.0%]
BoidFlockers	large	🔵 -0.0% [-0.6%, +0.6%]	🔵 +1.0% [+0.7%, +1.4%]

[quaquel]

Yes this should do it.

Can you make the PR text a bit more descriptive by linking it to the previous PR? I'll merge later today.

Thanks for the quick fix!

[Sahil-Chhoker]

Jan, I’d also love to hear why caching doesn’t work with generators when you have time.

[quaquel]

The cache returns the same generator object. This object is exhausted after the first iteration, so subsequent iterations never happen.