Skip to content

Commit

Permalink
Remove FIXME and fix imports
Browse files Browse the repository at this point in the history
  • Loading branch information
titodalcanton committed Dec 18, 2024
1 parent 1892c60 commit 53f00b4
Showing 1 changed file with 1 addition and 7 deletions.
8 changes: 1 addition & 7 deletions pycbc/distributions/sky_location.py
Original file line number Diff line number Diff line change
Expand Up @@ -20,8 +20,6 @@
import logging
import warnings
import numpy


from scipy.spatial.transform import Rotation

from pycbc.distributions import angular
Expand All @@ -30,8 +28,6 @@
from pycbc.types import angle_as_radians
from pycbc.libutils import import_optional

mhealpy = import_optional('mhealpy')


logger = logging.getLogger('pycbc.distributions.sky_location')

Expand Down Expand Up @@ -186,6 +182,7 @@ class HealpixSky:
def __init__(self, **params):
# Read the map file.
file_name = params['healpix_file']
mhealpy = import_optional('mhealpy')
self.healpix_map = mhealpy.HealpixMap.read_map(file_name)

# Get the probabilities at each pixel.
Expand Down Expand Up @@ -234,9 +231,6 @@ def pixel_corners(self, indices):
more HEALPix pixels. Dimension 0 is the pixel index, 1 is the Cartesian
coordinate, 2 is the corner index.
"""
# FIXME boundaries() works for an input array and is fast, but only for
# single-resolution maps. It raises an exception with MOC maps. Another
# bugfix for mhealpy?
return numpy.array(
[self.healpix_map.boundaries(pi, step=1) for pi in indices]
)
Expand Down

0 comments on commit 53f00b4

Please sign in to comment.