Opened 7 years ago

Closed 6 years ago

#364 closed defect (fixed)

Possible inconsistency in Poly_GaussCoil model

Reported by: gonzalezm Owned by: smk78
Priority: blocker Milestone: SasView 4.0.0
Component: SasView Keywords:
Cc: Work Package: SasView Documentation


Klaus Huber has noted that the polydisperse Gaussian coil model (in the shape independent category) contains already a model parameter (poly_m) that is related to the polydispersity. Therefore he considers that the user should
not have the option to activate also the polydispersity for the radius of gyration.

Change History (8)

comment:1 Changed 7 years ago by smk78

The poly_m parameter is related to the width of the chain length distribution (if poly_m=0 all chains are the same length). Clearly the chain length has a bearing on Rg since Rg=(n+1)-2 x [SUM(i,j<n) (Rij)2] where Rij is the distance between the i'th and j'th atoms in the chain. This is the basis of Klaus' point.

However, the chain length distribution does not specify the conformational freedom of the chains; chains of the same length could have different Rg's (consider swollen vs collapsed coils). So I would argue that you CAN have a distribution of Rg's as well.

comment:2 Changed 7 years ago by butler

While I agree that Steve is correct that Rg is affected by more than the Molecular weight distribution, I Think Klaus is correct in the sense that a polydispersity added to Rg would be 100% correlated to the poly_m parameter (and would be like fitting both the scale and sld difference).

That said the question I suppose is whehter we prevent Rg from having polidispersity added or not. I note that we do not prevent people from letting scale and delta(sld) vary simultaneously…. which in turn goes back to th philosophical discussion of how much you actively prevent people from doing what you think is wrong vs just warning - with respect to "warning" we can add that to the documnentation for sure!

comment:3 Changed 6 years ago by butler

  • Milestone changed from SasView Next Release +1 to SasView 4.0.0

comment:4 Changed 6 years ago by butler

  • Work Package changed from SasView Bug Fixing to SasView Documentation

comment:5 Changed 6 years ago by butler

  • Priority changed from major to blocker

Suggest we add some notes along these lines to the model documentation and close this ticket. As such it becomes part of the verify all model documentation core activity of release 4.0 and thus moving to this to blocker.

comment:6 Changed 6 years ago by butler

  • Owner set to butler
  • Status changed from new to accepted

comment:7 Changed 6 years ago by butler

  • Owner changed from butler to smk78
  • Status changed from accepted to assigned

comment:8 Changed 6 years ago by butler

  • Resolution set to fixed
  • Status changed from assigned to closed

The sasmodels implementation of this model does not in fact allow for polydispersity on Rg at the moment as it was re-written by Steve King. If he enables PD on the Rg he will change the documentation to reflect that. Thus this ticket is essentially addressed for now.

Note: See TracTickets for help on using tickets.