Skip to content

Commit

Permalink
Remove no longer relevant notes about isotropic grow
Browse files Browse the repository at this point in the history
The isotropic footprint growth metric has not been "slow" since about
2015 (see DM-1128 and DM-2787).
  • Loading branch information
parejkoj committed Aug 7, 2023
1 parent 974b69c commit bb91174
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 2 deletions.
2 changes: 1 addition & 1 deletion include/lsst/afw/detection/FootprintSet.h
Original file line number Diff line number Diff line change
Expand Up @@ -224,7 +224,7 @@ class FootprintSet {
* @param rhs the Footprints to merge
* @param tGrow No. of pixels to grow this Footprints
* @param rGrow No. of pixels to grow rhs Footprints
* @param isotropic Use (expensive) isotropic grow
* @param isotropic Grow isotropically (as opposed to a Manhattan metric)
*/
void merge(FootprintSet const& rhs, int tGrow = 0, int rGrow = 0, bool isotropic = true);

Expand Down
1 change: 0 additions & 1 deletion src/detection/FootprintSet.cc
Original file line number Diff line number Diff line change
Expand Up @@ -222,7 +222,6 @@ FootprintSet mergeFootprintSets(FootprintSet const &lhs, // the FootprintSe
// The isXXX routines return <isset, value>
bool const circular = ctrl.isCircular().first && ctrl.isCircular().second;
bool const isotropic = ctrl.isIsotropic().second; // isotropic grow as opposed to a Manhattan metric
// n.b. Isotropic grows are significantly slower
bool const left = ctrl.isLeft().first && ctrl.isLeft().second;
bool const right = ctrl.isRight().first && ctrl.isRight().second;
bool const up = ctrl.isUp().first && ctrl.isUp().second;
Expand Down

0 comments on commit bb91174

Please sign in to comment.