EcoJulia / NeutralLandscapes.jl

Generation of neutral landscapes in Julia.
https://ecojulia.github.io/NeutralLandscapes.jl/dev/
MIT License
16 stars 2 forks source link

fix view shape for perlin #36

Closed rafaqz closed 3 years ago

rafaqz commented 3 years ago

What the pull request does
Fixes row/ col order in views into the temp square array

Type of change
Please indicate the relevant option(s)

Checklist

rafaqz commented 3 years ago

We probably need to run all the algorithms for rectangle and square outputs of a few sizes and mix of parameters, not necessarily testing the output just that it works.

tpoisot commented 3 years ago

Can you add some tests on combinations of sizes to make sure that it marches?

rafaqz commented 3 years ago

I was thinking we should just do that for all of the methods here in one place. Just loop over some sizes/shapes and run them all. I'll look at writing that.

rafaqz commented 3 years ago

Ok tests are in the other PR as they may need separate discussion. However this needs to merge for them to pass! I didnt think that through properly...

mkborregaard commented 3 years ago

Just merge when you feel confident. We still have 0 users :-)