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
Let's discuss the issue raised in #378. Namely, abel/__init__.py imports
from .transformimportTransformfrom .tools.centerimportcenter_image
so that users can call the short forms abel.Transform and abel.center_image, without specifying corresponding submodules.
I guess, everybody agrees that abel.Transform is desirable, as it gives the simplest unified access to basic functionality, but the opinions regarding abel.center_image were different. Moreover, the latter wasn't documented anywhere.
So the question is whether we should leave/recommend only abel.Transform and deprecate abel.center_image (to be removed completely later), or to document abel.center_image and recommend it for basic tasks. If we incline towards the second option, then it might be also a good idea to add
from .tools.circularizeimportcircularize_image
because it's also quite useful (sometimes even necessary) in practice and, unlike center_image, is not available through Transform options.
The text was updated successfully, but these errors were encountered:
Let's discuss the issue raised in #378. Namely,
abel/__init__.py
importsso that users can call the short forms
abel.Transform
andabel.center_image
, without specifying corresponding submodules.I guess, everybody agrees that
abel.Transform
is desirable, as it gives the simplest unified access to basic functionality, but the opinions regardingabel.center_image
were different. Moreover, the latter wasn't documented anywhere.So the question is whether we should leave/recommend only
abel.Transform
and deprecateabel.center_image
(to be removed completely later), or to documentabel.center_image
and recommend it for basic tasks. If we incline towards the second option, then it might be also a good idea to addbecause it's also quite useful (sometimes even necessary) in practice and, unlike
center_image
, is not available throughTransform
options.The text was updated successfully, but these errors were encountered: