Fix ragged array warning in geomask.circle_boundaries()

desihub / desitarget

DESI Targeting

BSD 3-Clause "New" or "Revised" License

18 stars 23 forks source link

Fix ragged array warning in geomask.circle_boundaries() #781

Closed geordie666 closed 2 years ago

geordie666 commented 2 years ago

This PR fixes a ragged array deprecation warning in geomask.circle_boundaries().

The fix was to convert the ragged arrays to dtype=object while being careful to convert them back to dtype=float when needed.

This addresses #779.

geordie666 commented 2 years ago

@sbailey: If possible I'd like to merge this soon to include it in the tag for running MTL reprocessing (i.e. I'm about to merger #780 and tag).

coveralls commented 2 years ago

Coverage remained the same at 56.725% when pulling 23bd6d12fb1e40380121f67e375279398571b403 on ADM-offrapos into 303be12d9af7e7a85a27db112029fb7dc9c628b0 on master.