-
Notifications
You must be signed in to change notification settings - Fork 30
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
default effective radius type (Trac #1201) #191
Comments
Trac update at Hmm this is going to be tricky. Suspect that for backwards compatibility 5.0 ought to default to doing what 4.2 would do, thus as Paul K says to make sure that effective radius defaults to the one used by 4.2 This would reduce or perhaps remove the need for conversions. Would it help, before we get too far along with this, to change the default type of 0 to mean use ER from P(Q) rather than take a user supplied value? |
Trac update at The code in product.py needs to know whether to use the user value or the computed value independent of model. I'm guessing the GUI code also needs to know, given that it places the computed value into the user value slot when a computed value is requested. So best to leave that as option 0. Easiest is to set the default to 1 and make sure that the first effective radius is the one defined in sasview 4.1. |
Trac update at
to:
to:
A lot of the old models use some function which is perhaps the average radius of the equivalent cylinder, but which is not implemented for the new models. There are a couple of models which have an equivalent mode different from 1. There are a number of models that did not have ER before, but those ones won't be used with structure factors so it doesn't matter to the loader what they used to be. The new models all use the outer radius equivalent sphere, except mono gauss coil which uses Rg. beta_approx branch radius_effective_mode value for the old style ER function. Uses -- for not implemented and N/A for no equivalent. ||= Model =||= R_eff mode =||= old ER =|| The old cylinder ER expression is as follows:
I don't see it in [Pedersen 1997], and don't know where it comes from. Maybe [Fournet 1951]? Parallelepipeds used the long axis to define the equivalent cylinder. Would adding Rg for everything be useful? This is yet another |
Trac update at
In changeset 29c455a:
|
In sasmodels, all platforms.
For the beta_approx branch we are using a default of 0 as the effective radius type (i.e., use the value from the parameter table instead of the form factor calculation) rather than whatever the model was returning before.
Do we want this as the default?
Or should we set the default to 1 and make sure that is the "best" effective radius to use when computing P@S?
There is also an issue with reloading a saved P@S fit from 4.2 and earlier, which all implicitly use old ER radius type.
Do we try to set the equivalent effective radius type parameter in the model?
Or do we let it silently use the default value for radius_effective from S?
Note: if the old ER is not the first effective radius type for any model then conversion with need a table for each model giving the number to use. Not difficult, but a detail to test if we decide to support correct reloading of old P@S models.
Migrated from http://trac.sasview.org/ticket/1201
The text was updated successfully, but these errors were encountered: