Skip to content

Commit 255c52c

Browse files
committed
refactor(McModel): common dict for composed parameters
1 parent 7488477 commit 255c52c

File tree

1 file changed

+4
-5
lines changed

1 file changed

+4
-5
lines changed

src/mcsas3/mc_model.py

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -380,6 +380,7 @@ def checkSettings(self) -> None:
380380

381381
def calcModelIV(self, parameters: dict) -> Tuple[np.ndarray, np.ndarray]:
382382
# moved from McCore
383+
kernelParams = dict(self.staticParameters, **parameters)
383384
if (self.modelName.lower() != "sim") and (self.modelName.lower() != "mcsas_sphere"):
384385
# Fsq has been checked with Paul Kienzle, is the part in the square brackets squared
385386
# as in this equation (http://www.sasview.org/docs/user/models/sphere.html).
@@ -394,9 +395,7 @@ def calcModelIV(self, parameters: dict) -> Tuple[np.ndarray, np.ndarray]:
394395
self.kernel, (sasmodels.product.ProductKernel, sasmodels.mixture.MixtureKernel)
395396
):
396397
# call_Fq not available
397-
Fsq = sasmodels.direct_model.call_kernel(
398-
self.kernel, dict(self.staticParameters, **parameters)
399-
)
398+
Fsq = sasmodels.direct_model.call_kernel(self.kernel, kernelParams)
400399
try:
401400
V_shell = self.kernel.results()["volume"]
402401
except KeyError:
@@ -406,10 +405,10 @@ def calcModelIV(self, parameters: dict) -> Tuple[np.ndarray, np.ndarray]:
406405
Fsq = Fsq * V_shell
407406
else:
408407
F, Fsq, R_eff, V_shell, V_ratio = sasmodels.direct_model.call_Fq(
409-
self.kernel, dict(self.staticParameters, **parameters)
408+
self.kernel, kernelParams
410409
)
411410
else:
412-
Fsq, V_shell = self.kernel(**dict(self.staticParameters, **parameters))
411+
Fsq, V_shell = self.kernel(**kernelParams)
413412
# modelIntensity = Fsq/V_shell
414413
# modelVolume = V_shell
415414

0 commit comments

Comments
 (0)