cosmodesi / pyrecon

package for BAO reconstruction
BSD 3-Clause "New" or "Revised" License
9 stars 7 forks source link

allow array boxsize from positions #13

Closed seshnadathur closed 1 year ago

seshnadathur commented 1 year ago

Set boxsize to be an array when it is determined from positions, to allow for automatic creation of non-cubic boxes. Non-cubic boxes are allowed when specified by the user, but currently when automatically determining boxsize from positions the box is taken to be a cube, which is rather wasteful for most survey geometries.

seshnadathur commented 1 year ago

@adematti I merged this but maybe that was premature, have a look and revert if necessary

adematti commented 1 year ago

I think this is fine! I will update the NERSC environment.