Skip to content

Commit

Permalink
Change transfer of acqs_init cols into self
Browse files Browse the repository at this point in the history
Somewhere between numpy 1.12 and 1.15 the
behavior of argsort changed so that
np.argsort([0] * 20) might not be stable.
Table was relying on that (upstream fix needed),
but in the meantime adding columns like this is
now efficient in astropy Table.
  • Loading branch information
taldcroft committed May 13, 2020
1 parent 3101d66 commit 3f9e853
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion proseco/acq.py
Original file line number Diff line number Diff line change
Expand Up @@ -628,7 +628,8 @@ def get_initial_catalog(self):
acqs_init = cand_acqs[acq_indices]

# Transfer to acqs (which at this point is an empty table)
self.add_columns(acqs_init.columns.values())
for col in acqs_init.itercols():
self[col.info.name] = col

def calc_p_brightest(self, acq, box_size, man_err=0, bgd=0):
"""
Expand Down

0 comments on commit 3f9e853

Please sign in to comment.