You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The method desiutil.dust._Hemisphere.ebv(self, l, b, interpolate) triggers style warnings since l is considered a potentially ambiguous variable name. The suggested replacements are gal_l and gal_b, which are both more informative and unambiguous. However, since it is possible in principle that .ebv() is called with .ebv(l=..., b=..., ...), this is a potential API breaking change, so this should be applied to a future desiutil 4.0.
The text was updated successfully, but these errors were encountered:
I did a search for calls of .ebv(). I found no instances where .ebv() is called with l and b set as keywords. Furthermore, the .ebv() method that is proposed to be changed is actually wrapped by other .ebv() methods and/or the desiutil.dust.ebv() function.
The method
desiutil.dust._Hemisphere.ebv(self, l, b, interpolate)
triggers style warnings sincel
is considered a potentially ambiguous variable name. The suggested replacements aregal_l
andgal_b
, which are both more informative and unambiguous. However, since it is possible in principle that.ebv()
is called with.ebv(l=..., b=..., ...)
, this is a potential API breaking change, so this should be applied to a future desiutil 4.0.The text was updated successfully, but these errors were encountered: