Skip to content

Commit

Permalink
correct concatenation of objects in case we move to multiple streams
Browse files Browse the repository at this point in the history
  • Loading branch information
geordie666 committed Apr 1, 2024
1 parent 6f3e99f commit e4e9914
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions py/desitarget/streams/cuts.py
Expand Up @@ -194,7 +194,7 @@ def set_target_bits(objs, stream_names=["GD1"]):
Parameters
----------
objects : :class:`~numpy.ndarray`
objs : :class:`~numpy.ndarray`
numpy structured array with UPPERCASE columns needed for
stream target selection. See, e.g.,
:func:`~desitarget.stream.cuts.is_in_GD1` for column names.
Expand Down Expand Up @@ -291,7 +291,7 @@ def select_targets(swdir, stream_names=["GD1"], readperstream=True,
objs = read_data_per_stream(swdir, rapol, decpol, ra_ref, mind, maxd,
stream_name,
addnors=addnors, readcache=readcache)
allobjs.append(objs)
allobjs.append(objs)
objects = np.concatenate(allobjs)
else:
# ADM --TODO-- write loop across sweeps instead of streams.
Expand All @@ -301,7 +301,7 @@ def select_targets(swdir, stream_names=["GD1"], readperstream=True,

# ADM process the targets.
desi_target, bgs_target, mws_target, scnd_target = set_target_bits(
objs, stream_names=[stream_name])
objects, stream_names=stream_names)

# ADM finalize the targets.
# ADM anything with DESI_TARGET !=0 is truly a target.
Expand Down

0 comments on commit e4e9914

Please sign in to comment.