Skip to content

Commit

Permalink
Removing float conversion and adding docstring for mean_ra and
Browse files Browse the repository at this point in the history
mean_dec variables
  • Loading branch information
jacquot committed Dec 17, 2024
1 parent 0e0b138 commit b737c3c
Showing 1 changed file with 7 additions and 5 deletions.
12 changes: 7 additions & 5 deletions pycbc/distributions/sky_location.py
Original file line number Diff line number Diff line change
Expand Up @@ -69,9 +69,11 @@ class FisherSky:
Parameters
----------
mean_ra: float
RA of the center of the distribution.
RA of the center of the distribution. Use the rad or deg suffix to
specify units, otherwise radians are assumed.
mean_dec: float
Declination of the center of the distribution.
Declination of the center of the distribution. Use the rad or deg
suffix to specify units, otherwise radians are assumed.
sigma: float
Spread of the distribution. For the precise interpretation, see Eq 8
of `Briggs et al 1999 ApJS 122 503`_. This should be smaller than
Expand Down Expand Up @@ -124,9 +126,9 @@ def from_config(cls, cp, section, variable_args):
"Not all parameters used by this distribution "
"included in tag portion of section name"
)
mean_ra = float(cp.get_opt_tag(section, 'mean_ra', tag))
mean_dec = float(cp.get_opt_tag(section, 'mean_dec', tag))
sigma = float(cp.get_opt_tag(section, 'sigma', tag))
mean_ra = cp.get_opt_tag(section, 'mean_ra', tag)
mean_dec = cp.get_opt_tag(section, 'mean_dec', tag)
sigma = cp.get_opt_tag(section, 'sigma', tag)
return cls(
mean_ra=mean_ra,
mean_dec=mean_dec,
Expand Down

0 comments on commit b737c3c

Please sign in to comment.