Skip to content

Commit 7bb8e7e

Browse files
committed
Restore to previous logic and order in setting params
1 parent e20ad24 commit 7bb8e7e

File tree

1 file changed

+13
-10
lines changed

1 file changed

+13
-10
lines changed

radio_beam/beam.py

+13-10
Original file line numberDiff line numberDiff line change
@@ -81,20 +81,23 @@ def __new__(cls, major=None, minor=None, pa=None, area=None,
8181
minor = rad * SIGMA_TO_FWHM
8282
pa = 0.0 * u.deg
8383

84-
else:
85-
# give specified values priority
86-
84+
# give specified values priority
85+
if major is not None:
8786
major = _set_default_unit("major", major, default_unit, equiv_unit=u.deg)
87+
88+
if pa is not None:
8889
pa = _set_default_unit("pa", pa, default_unit, equiv_unit=u.deg)
90+
else:
91+
pa = 0 * u.deg
8992

90-
# some sensible defaults
91-
if minor is None:
92-
minor = major
93-
else:
94-
minor = _set_default_unit("minor", minor, default_unit, equiv_unit=u.deg)
93+
# some sensible defaults
94+
if minor is None:
95+
minor = major
96+
else:
97+
minor = _set_default_unit("minor", minor, default_unit, equiv_unit=u.deg)
9598

96-
if minor > major:
97-
raise ValueError("Minor axis greater than major axis.")
99+
if minor > major:
100+
raise ValueError("Minor axis greater than major axis.")
98101

99102
self = super(Beam, cls).__new__(cls, _to_area(major,minor).value, u.sr)
100103
self._major = major

0 commit comments

Comments
 (0)