DESI-UR / VAST

Void Analysis Software Toolkit
https://vast.readthedocs.io/en/latest/
BSD 3-Clause "New" or "Revised" License
9 stars 8 forks source link

VF finding galaxies in holes #82

Closed kadglass closed 1 year ago

kadglass commented 2 years ago

In the unit test for VoidFinder, found a test case where a hole was being found which had numerous wall galaxies significantly inside it. It dissapears/reappears with a comment out for a final-extra-check in find_next_galaxy() possibly indicating a logic error somewhere. Possibly related to main_algorithm() checking potential bad unit vectors when finding the 4th bounding galaxy.

kadglass commented 1 year ago

Fixed with PR #91