Skip to content
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

Fix volume normalisation on Raspberry model (Trac #845) #140

Open
ajj opened this issue Mar 30, 2019 · 4 comments
Open

Fix volume normalisation on Raspberry model (Trac #845) #140

ajj opened this issue Mar 30, 2019 · 4 comments

Comments

@ajj
Copy link
Member

ajj commented Mar 30, 2019

As noted in SasView/sasview#912 the raspberry model needs to be doing volume calculation outside Iq function to get polydispersity working properly.

Migrated from http://trac.sasview.org/ticket/845

{
    "status": "accepted",
    "changetime": "2018-02-13T02:57:55",
    "_ts": "2018-02-13 02:57:55.022121+00:00",
    "description": "As noted in #815 the raspberry model needs to be doing volume calculation outside Iq function to get polydispersity working properly.",
    "reporter": "ajj",
    "cc": "",
    "resolution": "",
    "workpackage": "SasView Bug Fixing",
    "time": "2017-02-07T15:15:35",
    "component": "sasmodels",
    "summary": "Fix volume normalisation on Raspberry model",
    "priority": "major",
    "keywords": "",
    "milestone": "SasView 4.3.0",
    "owner": "ajj",
    "type": "defect"
}
@pkienzle
Copy link
Contributor

Trac update at 2017/02/07 19:17:41: pkienzle commented:

Should use the square() function for complex calculations:

     sfSS = psiS*psiS*sinc(q*(rL+deltaS*rS))*sinc(q*(rL+deltaS*rS));

becomes:

     sfSS = square(psiS*sinc(q*(rL+deltaS*rS)));

This makes life easier for the compiler since it doesn't have to figure out that the two subexpressions are identical, and saves four multiplies and a branch for each q.

Note that we now have sas_sinx_x as an available function instead of sinc.

@ajj
Copy link
Member Author

ajj commented Mar 30, 2019

Trac update at 2017/04/04 21:52:24:

  • ajj changed owner from "" to "ajj"
  • ajj changed status from "new" to "accepted"

@ajj
Copy link
Member Author

ajj commented Mar 30, 2019

Trac update at 2017/04/09 13:26:03: ajj commented:

This model doesn't have a single form factor volume as it actually is the sum of two form factors - the raspberry, and the free small particles.

Need to add polydispersity loop directly in this model as the auto polydispersity cannot work.

Longer term, should we implement models like this as composition models?

@butlerpd
Copy link
Member

Trac update at 2018/02/13 02:57:55: butler changed milestone from "SasView 4.2.0" to "SasView 4.3.0"

@pkienzle pkienzle modified the milestones: SasView 4.3.0, sasmodels 1.0 Mar 31, 2019
@butlerpd butlerpd modified the milestones: sasmodels 1.0, sasmodels 1.1 Aug 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants