search

ajmendez / stomp

STOMP for astro
0 stars 0 forks source link

bug with FindPairs #6

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
What steps will reproduce the problem?
1. Run FindPairs on an IndexedTreeMap() with a large separation angle

Expected output:
A list of indices

Error given by stomp:
" Illegal theta value on x index: Lambda,Eta = -22.4984,-32.7357,theta = 117.229
  x = 164/143"

Original issue reported on code.google.com by menard.c...@gmail.com on 17 Jun 2010 at 6:38

GoogleCodeExporter commented 9 years ago 
The problem here stemmed from the XYBound method in the Pixel class.  This 
method is intended to return a list of indices for Pixels that are within some 
input angle theta of either the Pixel center or an input AngularCoordinate.  
The code failed because there was no allowance for the possibility that the 
input value of theta would be larger than the EtaOffset value in stomp_core.h.  
Now it checks against this and reacts accordingly.

Original comment by ryan.scr...@gmail.com on 21 Jun 2010 at 11:39