Merge branch 'main' into ADM-cols-mtl-hp

desihub · Sep 9, 2023 · 967b737 · 967b737
2 parents ec1dc5f + dd8ba1b
commit 967b737
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/doc/changes.rst b/doc/changes.rst
@@ -5,6 +5,7 @@ desitarget Change Log
 2.6.2 (unreleased)
 ------------------
 
+* Reproducible output from :func:`io.read_targets_in_hp()` [`PR #806`_].
 * Option to read subset of columns from HEALPix-split MTLs [`PR #805`_].
     * Specifically in :func:`io.read_mtl_in_hp()`.
     * Addresses `issue #802`_.
@@ -14,6 +15,7 @@ desitarget Change Log
 .. _`issue #802`: https://github.com/desihub/desitarget/issues/802
 .. _`issue #804`: https://github.com/desihub/desitarget/issues/804
 .. _`PR #805`: https://github.com/desihub/desitarget/pull/805
+.. _`PR #806`: https://github.com/desihub/desitarget/pull/806
 
 2.6.1 (2023-08-23)
 ------------------

diff --git a/py/desitarget/io.py b/py/desitarget/io.py
@@ -3366,7 +3366,7 @@ def read_targets_in_hp(hpdirname, nside, pixlist, columns=None, header=False,
         filepixlist = filepixlist[isindict]
 
         # ADM make sure each file is only read once.
-        infiles = set([filedict[pix] for pix in filepixlist])
+        infiles = sorted(set([filedict[pix] for pix in filepixlist]))
 
         # ADM read in the files and concatenate the resulting targets.
         targets = []