Skip to content

Commit fca22ce

Browse files
Jammy2211Jammy2211
authored andcommitted
refactor and simplify SettingsInversion
1 parent a03a6b4 commit fca22ce

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

42 files changed

+209
-233
lines changed

autoarray/__init__.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@
2727
from .inversion.pixelization import mesh
2828
from .inversion.pixelization import image_mesh
2929
from .inversion import regularization as reg
30-
from .inversion.inversion.settings import SettingsInversion
30+
from .settings import Settings
3131
from .inversion.inversion.abstract import AbstractInversion
3232
from .inversion.regularization.abstract import AbstractRegularization
3333
from .inversion.inversion.factory import inversion_from as Inversion
@@ -40,8 +40,8 @@
4040
from .inversion.pixelization.mappers.rectangular_uniform import MapperRectangularUniform
4141
from .inversion.pixelization.image_mesh.abstract import AbstractImageMesh
4242
from .inversion.pixelization.mesh.abstract import AbstractMesh
43-
from .inversion.pixelization.mesh_grid.rectangular_2d import Mesh2DRectangular
44-
from .inversion.pixelization.mesh_grid.delaunay_2d import Mesh2DDelaunay
43+
from .inversion.pixelization.mesh_grid.rectangular import Mesh2DRectangular
44+
from .inversion.pixelization.mesh_grid.delaunay import Mesh2DDelaunay
4545
from .inversion.inversion.imaging.mapping import InversionImagingMapping
4646
from .inversion.inversion.imaging.sparse import InversionImagingSparse
4747
from .inversion.inversion.imaging.inversion_imaging_util import ImagingSparseOperator
@@ -77,8 +77,8 @@
7777
from .structures.grids.uniform_2d import Grid2D
7878
from .operators.over_sampling.over_sampler import OverSampler
7979
from .structures.grids.irregular_2d import Grid2DIrregular
80-
from .inversion.pixelization.mesh_grid.rectangular_2d import Mesh2DRectangular
81-
from .inversion.pixelization.mesh_grid.delaunay_2d import Mesh2DDelaunay
80+
from .inversion.pixelization.mesh_grid.rectangular import Mesh2DRectangular
81+
from .inversion.pixelization.mesh_grid.delaunay import Mesh2DDelaunay
8282
from .structures.arrays.kernel_2d import Kernel2D
8383
from .structures.vectors.uniform import VectorYX2D
8484
from .structures.vectors.irregular import VectorYX2DIrregular

autoarray/config/general.yaml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@ inversion:
66
check_reconstruction: true # If True, the inversion's reconstruction is checked to ensure the solution of a meshs's mapper is not an invalid solution where the values are all the same.
77
use_positive_only_solver: true # If True, inversion's use a positive-only linear algebra solver by default, which is slower but prevents unphysical negative values in the reconstructed solutuion.
88
no_regularization_add_to_curvature_diag_value : 1.0e-3 # The default value added to the curvature matrix's diagonal when regularization is not applied to a linear object, which prevents inversion's failing due to the matrix being singular.
9-
positive_only_uses_p_initial: true # If True, the positive-only solver of an inversion's uses an initial guess of the reconstructed data's values as which values should be positive, speeding up the solver.
109
use_border_relocator: false # If True, by default a pixelization's border is used to relocate all pixels outside its border to the border.
1110
reconstruction_vmax_factor: 0.5 # Plots of an Inversion's reconstruction use the reconstructed data's bright value multiplied by this factor.
1211
numba:

autoarray/inversion/inversion/abstract.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
from autoarray.inversion.linear_obj.linear_obj import LinearObj
1212
from autoarray.inversion.pixelization.mappers.abstract import AbstractMapper
1313
from autoarray.inversion.regularization.abstract import AbstractRegularization
14-
from autoarray.inversion.inversion.settings import SettingsInversion
14+
from autoarray.settings import Settings
1515
from autoarray.preloads import Preloads
1616
from autoarray.structures.arrays.uniform_2d import Array2D
1717
from autoarray.structures.grids.irregular_2d import Grid2DIrregular
@@ -26,7 +26,7 @@ def __init__(
2626
self,
2727
dataset: Union[Imaging, Interferometer, DatasetInterface],
2828
linear_obj_list: List[LinearObj],
29-
settings: SettingsInversion = SettingsInversion(),
29+
settings: Settings = None,
3030
preloads: Preloads = None,
3131
xp=np,
3232
):
@@ -72,7 +72,7 @@ def __init__(
7272

7373
self.linear_obj_list = linear_obj_list
7474

75-
self.settings = settings
75+
self.settings = settings or Settings()
7676

7777
self.preloads = preloads or Preloads()
7878

autoarray/inversion/inversion/factory.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -23,15 +23,15 @@
2323
from autoarray.inversion.inversion.imaging.sparse import (
2424
InversionImagingSparse,
2525
)
26-
from autoarray.inversion.inversion.settings import SettingsInversion
26+
from autoarray.settings import Settings
2727
from autoarray.preloads import Preloads
2828
from autoarray.structures.arrays.uniform_2d import Array2D
2929

3030

3131
def inversion_from(
3232
dataset: Union[Imaging, Interferometer, DatasetInterface],
3333
linear_obj_list: List[LinearObj],
34-
settings: SettingsInversion = SettingsInversion(),
34+
settings: Settings = None,
3535
preloads: Preloads = None,
3636
xp=np,
3737
):
@@ -80,7 +80,7 @@ def inversion_from(
8080
def inversion_imaging_from(
8181
dataset,
8282
linear_obj_list: List[LinearObj],
83-
settings: SettingsInversion = SettingsInversion(),
83+
settings: Settings = None,
8484
preloads: Preloads = None,
8585
xp=np,
8686
):
@@ -157,7 +157,7 @@ def inversion_imaging_from(
157157
def inversion_interferometer_from(
158158
dataset: Union[Interferometer, DatasetInterface],
159159
linear_obj_list: List[LinearObj],
160-
settings: SettingsInversion = SettingsInversion(),
160+
settings: Settings = None,
161161
xp=np,
162162
):
163163
"""

autoarray/inversion/inversion/imaging/abstract.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
from autoarray.inversion.pixelization.mappers.abstract import AbstractMapper
88
from autoarray.inversion.inversion.abstract import AbstractInversion
99
from autoarray.inversion.linear_obj.linear_obj import LinearObj
10-
from autoarray.inversion.inversion.settings import SettingsInversion
10+
from autoarray.settings import Settings
1111
from autoarray.preloads import Preloads
1212

1313
from autoarray.inversion.inversion.imaging import inversion_imaging_util
@@ -18,7 +18,7 @@ def __init__(
1818
self,
1919
dataset: Union[Imaging, DatasetInterface],
2020
linear_obj_list: List[LinearObj],
21-
settings: SettingsInversion = SettingsInversion(),
21+
settings: Settings = None,
2222
preloads: Preloads = None,
2323
xp=np,
2424
):

autoarray/inversion/inversion/imaging/mapping.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88
from autoarray.inversion.inversion.imaging.abstract import AbstractInversionImaging
99
from autoarray.inversion.linear_obj.linear_obj import LinearObj
1010
from autoarray.inversion.pixelization.mappers.abstract import AbstractMapper
11-
from autoarray.inversion.inversion.settings import SettingsInversion
11+
from autoarray.settings import Settings
1212
from autoarray.preloads import Preloads
1313
from autoarray.structures.arrays.uniform_2d import Array2D
1414

@@ -21,7 +21,7 @@ def __init__(
2121
self,
2222
dataset: Union[Imaging, DatasetInterface],
2323
linear_obj_list: List[LinearObj],
24-
settings: SettingsInversion = SettingsInversion(),
24+
settings: Settings = None,
2525
preloads: Preloads = None,
2626
xp=np,
2727
):

autoarray/inversion/inversion/imaging/sparse.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
from autoarray.inversion.inversion.dataset_interface import DatasetInterface
88
from autoarray.inversion.inversion.imaging.abstract import AbstractInversionImaging
99
from autoarray.inversion.linear_obj.linear_obj import LinearObj
10-
from autoarray.inversion.inversion.settings import SettingsInversion
10+
from autoarray.settings import Settings
1111
from autoarray.inversion.linear_obj.func_list import AbstractLinearObjFuncList
1212
from autoarray.inversion.pixelization.mappers.abstract import AbstractMapper
1313
from autoarray.preloads import Preloads
@@ -21,7 +21,7 @@ def __init__(
2121
self,
2222
dataset: Union[Imaging, DatasetInterface],
2323
linear_obj_list: List[LinearObj],
24-
settings: SettingsInversion = SettingsInversion(),
24+
settings: Settings = None,
2525
preloads: Preloads = None,
2626
xp=np,
2727
):

autoarray/inversion/inversion/imaging_numba/sparse.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
from autoarray.inversion.inversion.dataset_interface import DatasetInterface
88
from autoarray.inversion.inversion.imaging.abstract import AbstractInversionImaging
99
from autoarray.inversion.linear_obj.linear_obj import LinearObj
10-
from autoarray.inversion.inversion.settings import SettingsInversion
10+
from autoarray.settings import Settings
1111
from autoarray.inversion.linear_obj.func_list import AbstractLinearObjFuncList
1212
from autoarray.inversion.pixelization.mappers.abstract import AbstractMapper
1313
from autoarray.preloads import Preloads
@@ -21,7 +21,7 @@ def __init__(
2121
self,
2222
dataset: Union[Imaging, DatasetInterface],
2323
linear_obj_list: List[LinearObj],
24-
settings: SettingsInversion = SettingsInversion(),
24+
settings: Settings = None,
2525
preloads: Preloads = None,
2626
xp=np,
2727
):

autoarray/inversion/inversion/interferometer/abstract.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
from autoarray.inversion.inversion.abstract import AbstractInversion
77
from autoarray.mask.mask_2d import Mask2D
88
from autoarray.inversion.linear_obj.linear_obj import LinearObj
9-
from autoarray.inversion.inversion.settings import SettingsInversion
9+
from autoarray.settings import Settings
1010
from autoarray.structures.arrays.uniform_2d import Array2D
1111

1212
from autoarray.inversion.inversion import inversion_util
@@ -17,7 +17,7 @@ def __init__(
1717
self,
1818
dataset: Union[Interferometer, DatasetInterface],
1919
linear_obj_list: List[LinearObj],
20-
settings: SettingsInversion = SettingsInversion(),
20+
settings: Settings = None,
2121
xp=np,
2222
):
2323
"""

autoarray/inversion/inversion/interferometer/mapping.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
AbstractInversionInterferometer,
88
)
99
from autoarray.inversion.linear_obj.linear_obj import LinearObj
10-
from autoarray.inversion.inversion.settings import SettingsInversion
10+
from autoarray.settings import Settings
1111
from autoarray.structures.visibilities import Visibilities
1212

1313
from autoarray.inversion.inversion.interferometer import inversion_interferometer_util
@@ -19,7 +19,7 @@ def __init__(
1919
self,
2020
dataset: Union[Interferometer, DatasetInterface],
2121
linear_obj_list: List[LinearObj],
22-
settings: SettingsInversion = SettingsInversion(),
22+
settings: Settings = None,
2323
xp=np,
2424
):
2525
"""

0 commit comments

Comments
 (0)