astropy / astroplan

Observation planning package for astronomers – maintainer @bmorris3
https://astroplan.readthedocs.io/en/latest/
BSD 3-Clause "New" or "Revised" License
198 stars 109 forks source link

Respect the order of blocks with the same priority in PriorityScheduler #541

Closed michaelbaisch closed 1 year ago

michaelbaisch commented 1 year ago

Update np.argsort() to use mergesort instead of quicksort for consistent sorting of equal values

Fixes #397

bmorris3 commented 1 year ago

Tidy fix, @michaelbaisch! Would you mind writing a short test to verify that the new behavior acts as expected?

michaelbaisch commented 1 year ago

anything still missing here?

bmorris3 commented 1 year ago

All good, thank you @michaelbaisch! And sorry for the delay ⌛