Opened 6 years ago
Closed 6 years ago
#1148 closed enhancement (fixed)
Documentation for S(Q) models need updating
Reported by: | butler | Owned by: | butler |
---|---|---|---|
Priority: | critical | Milestone: | SasView 4.3.0 |
Component: | SasView | Keywords: | |
Cc: | Work Package: | SasView Documentation |
Description
The model documentations auto generates some of the documentation such as the example plot (using default parameters) and adds the scale and background parameters to the table.
For S(Q) however there is no scale and background term when using the model (a change from previous versions I believe) but the documentation still generates those leading to some user confusion.
Change History (6)
comment:1 Changed 6 years ago by butler
- Owner set to butler
- Status changed from new to assigned
comment:2 Changed 6 years ago by pkienzle
comment:3 Changed 6 years ago by butler
- Resolution set to fixed
- Status changed from assigned to closed
Fixed by commit6e45516f19370dc8d3a8ff3fa00959f6e27d9e19. Tested that it does as advertized. Was pushed directly to master rather than through a PR as was a rather simple fix to documentation and sasmodels cannot yet have builds for testing. Basically create a conditional block in generate.py so that "COMMON" parameters (i.e. those auto generated for every mode) are not added to the parameter table in the generated docs for structure factor models (but is for all others).
Now closing ticket
comment:4 Changed 6 years ago by smk78
- Milestone changed from SasView 4.2.0 to SasView 4.3.0
- Resolution fixed deleted
- Status changed from closed to reopened
- Type changed from defect to enhancement
- Work Package changed from SasView Bug Fixing to SasView Documentation
Re-opening this ticket as I just noticed that none of the S(Q) models actually describe the functions they are applying!
comment:5 Changed 6 years ago by pkienzle
The ticket description and possible the title need to be changed to reflect this new requirement.
comment:6 Changed 6 years ago by butler
- Resolution set to fixed
- Status changed from reopened to closed
Indeed this ticket was not about the lack of equation or other lack in the writeup but about the autogeneration doing something wrong. That was fixed in the commit listed. The title was probably not the best.
Going to reclose this ticket and open a new one as actually this is not a problem specific to SQ. I have found there are several models that do not have the equations listed …
Fixed in sasmodels. Close ticket when verified.