Opened 4 years ago
Last modified 3 years ago
#1107 new defect
Doubts about Onion Doc
Reported by: | Joachim Wuttke | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | SasView 4.3.0 |
Component: | sasmodels | Keywords: | |
Cc: | Work Package: | SasModels Model Issues |
Description
quite generally:
to facilitate citation and discussion, I'd suggest to number equations
in doc pages that contain lots of them.
concerning http://marketplace.sasview.org/models/73:
The introduction announces that the SLD of each shell may be »described
by an exponential, linear, or constant function«. In the following,
however, only only the exponential and the constant case are explicitly
covered. For a linear SLD, the reader is left with the hint that
the exponential function with small A is a good approximation.
rho_shell(r) is defined in terms of six parameters. One of them is redundant
since rho_in=B+C. The parameters B and C have no obvious physical meaning,
and they do not appear in the parameter list of Iq. Two equation blocks later,
the paremeter B is defined a second time, now in terms of rho_in, rho_out, and A.
This makes the logic very obscure.
I would suggest that rho_shell(r) be defined in terms of the parameters
A, rho_in, rho_out, r_in, r_out. Then one would introduce Delta t_shell,
B, and C as abbreviations, which makes them categorically distinct from
the API parameters.
- Joachim
Change History (4)
comment:1 Changed 4 years ago by richardh
comment:2 Changed 3 years ago by smk78
Joachim's comment about B & C not appearing in the parameter table is invalid because they are defined by parameters that do; as the docs clearly state.
But I have reworked the docs a little to try and make the different conditions a bit clearer.
His remaining comments are essentially suggesting a reparameterisation of fthe model which is not a documentation issue.
comment:3 Changed 3 years ago by smk78
- Work Package changed from SasView Bug Fixing to SasModels Model Issues
comment:4 Changed 3 years ago by pkienzle
There are equations for linear shells in spherical_sld:
I suspect there is a Taylor series for low qr that improves accuracy and handles q=0.
Don't know if they are valid for large steps, or if they are only an approximation.
I did work out the equations for sld linear increasing or decreasing spherical shells, so one day we could add them properly.
Richard