Skip to content

Commit d1e8e3b

Browse files
authored
Removes unused properties from the API (RascalSoftware#137)
* Removes unused properties from the API * Updates "define.h"
1 parent 147cbee commit d1e8e3b

File tree

6 files changed

+78
-283
lines changed

6 files changed

+78
-283
lines changed

RATapi/inputs.py

Lines changed: 0 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -142,9 +142,6 @@ def make_input(project: RATapi.Project, controls: RATapi.Controls) -> tuple[Prob
142142
[[element.min, element.max] for element in getattr(project, class_list)],
143143
)
144144

145-
# Use dummy value for qzshifts
146-
limits.qzshifts = []
147-
148145
if project.model == LayerModels.CustomXY:
149146
controls.calcSldDuringFit = True
150147

@@ -298,15 +295,13 @@ def make_problem(project: RATapi.Project) -> ProblemDefinition:
298295
problem.dataPresent = make_data_present(project)
299296
problem.dataLimits = data_limits
300297
problem.simulationLimits = simulation_limits
301-
problem.oilChiDataPresent = [0] * len(project.contrasts)
302298
problem.numberOfContrasts = len(project.contrasts)
303299
problem.geometry = project.geometry
304300
problem.useImaginary = project.absorption
305301
problem.repeatLayers = [[0, 1]] * len(project.contrasts) # This is marked as "to do" in RAT
306302
problem.contrastBackgroundParams = contrast_background_params
307303
problem.contrastBackgroundTypes = contrast_background_types
308304
problem.contrastBackgroundActions = [contrast.background_action for contrast in project.contrasts]
309-
problem.contrastQzshifts = [1] * len(project.contrasts) # This is marked as "to do" in RAT
310305
problem.contrastScalefactors = [
311306
project.scalefactors.index(contrast.scalefactor, True) for contrast in project.contrasts
312307
]
@@ -317,7 +312,6 @@ def make_problem(project: RATapi.Project) -> ProblemDefinition:
317312
problem.contrastResolutionTypes = contrast_resolution_types
318313

319314
problem.backgroundParams = [param.value for param in project.background_parameters]
320-
problem.qzshifts = [0.0]
321315
problem.scalefactors = [param.value for param in project.scalefactors]
322316
problem.bulkIns = [param.value for param in project.bulk_in]
323317
problem.bulkOuts = [param.value for param in project.bulk_out]
@@ -358,18 +352,6 @@ def make_problem(project: RATapi.Project) -> ProblemDefinition:
358352
for param in getattr(project, class_list)
359353
if param.fit
360354
]
361-
problem.otherParams = [
362-
param.value
363-
for class_list in RATapi.project.parameter_class_lists
364-
for param in getattr(project, class_list)
365-
if not param.fit
366-
]
367-
problem.otherLimits = [
368-
[param.min, param.max]
369-
for class_list in RATapi.project.parameter_class_lists
370-
for param in getattr(project, class_list)
371-
if not param.fit
372-
]
373355
problem.priorNames = [
374356
param.name for class_list in RATapi.project.parameter_class_lists for param in getattr(project, class_list)
375357
]
@@ -392,10 +374,6 @@ def make_problem(project: RATapi.Project) -> ProblemDefinition:
392374
problem.checks, parameter_field[class_list], [int(element.fit) for element in getattr(project, class_list)]
393375
)
394376

395-
# Use dummy values for qz shifts
396-
problem.names.qzshifts = []
397-
problem.checks.qzshifts = []
398-
399377
check_indices(problem)
400378

401379
return problem

cpp/includes/defines.h

Lines changed: 0 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -397,8 +397,6 @@ backgroundParams : np.ndarray[np.float]
397397
Limits for backgroundParams in the problem definition.
398398
scalefactors : np.ndarray[np.float]
399399
Limits for scalefactors in the problem definition.
400-
qzshifts : np.ndarray[np.float]
401-
Limits for qzshifts in the problem definition.
402400
bulkIns : np.ndarray[np.float]
403401
Limits for bulkIns in the problem definition.
404402
bulkOuts : np.ndarray[np.float]
@@ -413,7 +411,6 @@ struct Limits {
413411
py::array_t<real_T> params;
414412
py::array_t<real_T> backgroundParams;
415413
py::array_t<real_T> scalefactors;
416-
py::array_t<real_T> qzshifts;
417414
py::array_t<real_T> bulkIns;
418415
py::array_t<real_T> bulkOuts;
419416
py::array_t<real_T> resolutionParams;
@@ -431,8 +428,6 @@ backgroundParams : list
431428
Names of backgroundParams in the problem definition.
432429
scalefactors : list
433430
Names of scalefactors in the problem definition.
434-
qzshifts : list
435-
Names of qzshifts in the problem definition.
436431
bulkIns : list
437432
Names of bulkIns in the problem definition.
438433
bulkName: list
@@ -447,7 +442,6 @@ struct NameStore {
447442
py::list params;
448443
py::list backgroundParams;
449444
py::list scalefactors;
450-
py::list qzshifts;
451445
py::list bulkIns;
452446
py::list bulkOuts;
453447
py::list resolutionParams;
@@ -469,8 +463,6 @@ backgroundParams : np.ndarray[np.float]
469463
Non-zero values indicates which backgroundParams is fitted.
470464
scalefactors : np.ndarray[np.float]
471465
Non-zero values indicates which scalefactors is fitted.
472-
qzshifts : np.ndarray[np.float]
473-
Non-zero values indicates which qzshifts is fitted.
474466
bulkIns : np.ndarray[np.float]
475467
Non-zero values indicates which bulkIns is fitted.
476468
bulkOuts : np.ndarray[np.float]
@@ -484,7 +476,6 @@ domainRatios : np.ndarray[np.float]
484476
struct Checks {
485477
py::array_t<real_T> params;
486478
py::array_t<real_T> backgroundParams;
487-
py::array_t<real_T> qzshifts;
488479
py::array_t<real_T> scalefactors;
489480
py::array_t<real_T> bulkIns;
490481
py::array_t<real_T> bulkOuts;
@@ -508,8 +499,6 @@ dataLimits : list
508499
Data limits for each contrast.
509500
simulationLimits : list;
510501
Simulation for each contrast.
511-
oilChiDataPresent : np.ndarray[np.float]
512-
If ``dataPresent[i]`` is non-zero, then contrast ``i`` has oilChi data. This is currently not being used.
513502
numberOfContrasts : int
514503
Number of contrasts.
515504
geometry : str
@@ -525,8 +514,6 @@ contrastBackgroundTypes : list
525514
Background type for each contrast.
526515
contrastBackgroundActions : list
527516
Background action for each contrast.
528-
contrastQzshifts : np.ndarray[np.float]
529-
Indices of Qzshifts used for each contrast. This is currently not being used.
530517
contrastScalefactors : np.ndarray[np.float]
531518
Indices of scalefactors used for each contrast.
532519
contrastBulkIns : np.ndarray[np.float]
@@ -539,8 +526,6 @@ contrastResolutionTypes : list
539526
Resolution type for each contrast.
540527
backgroundParams : np.ndarray[np.float]
541528
Background parameter values.
542-
qzshifts : np.ndarray[np.float]
543-
Qzshift values. This currently not being used.
544529
scalefactors : np.ndarray[np.float]
545530
Scalefactors values.
546531
bulkIns : np.ndarray[np.float]
@@ -573,12 +558,8 @@ domainContrastLayers : list
573558
Indices of layers added to the model of each domain contrast.
574559
fitParams : np.ndarray[np.float]
575560
Values of fitted parameters.
576-
otherParams : np.ndarray[np.float]
577-
Values of non-fitted parameters.
578561
fitLimits : np.ndarray[np.float]
579562
Limits of fitted parameters.
580-
otherLimits : np.ndarray[np.float]
581-
Limits of non fitted parameters.
582563
priorNames : list
583564
Parameter names for for all parameters in the problem definition.
584565
priorValues : np.ndarray[np.float]
@@ -596,22 +577,19 @@ struct ProblemDefinition {
596577
py::array_t<real_T> dataPresent;
597578
py::list dataLimits;
598579
py::list simulationLimits;
599-
py::array_t<real_T> oilChiDataPresent;
600580
real_T numberOfContrasts;
601581
std::string geometry {};
602582
boolean_T useImaginary {};
603583
py::list repeatLayers;
604584
py::list contrastBackgroundParams;
605585
py::list contrastBackgroundTypes;
606586
py::list contrastBackgroundActions;
607-
py::array_t<real_T> contrastQzshifts;
608587
py::array_t<real_T> contrastScalefactors;
609588
py::array_t<real_T> contrastBulkIns;
610589
py::array_t<real_T> contrastBulkOuts;
611590
py::list contrastResolutionParams;
612591
py::list contrastResolutionTypes;
613592
py::array_t<real_T> backgroundParams;
614-
py::array_t<real_T> qzshifts;
615593
py::array_t<real_T> scalefactors;
616594
py::array_t<real_T> bulkIns;
617595
py::array_t<real_T> bulkOuts;
@@ -628,9 +606,7 @@ struct ProblemDefinition {
628606
real_T numberOfDomainContrasts {};
629607
py::list domainContrastLayers;
630608
py::array_t<real_T> fitParams;
631-
py::array_t<real_T> otherParams;
632609
py::array_t<real_T> fitLimits;
633-
py::array_t<real_T> otherLimits;
634610
py::list priorNames;
635611
py::array_t<real_T> priorValues;
636612
NameStore names;

0 commit comments

Comments
 (0)