Opened 6 years ago

Closed 6 years ago

#1128 closed defect (fixed)

AutoPlot generation for model documentation does not include background

Reported by: butler Owned by: pkienzle
Priority: major Milestone: SasView 4.2.0
Component: SasView Keywords:
Cc: Work Package: SasView Documentation

Description

Somewhere along the line in transitioning from backgrounds explicitly being defined in models and being managed by the sasmodels infrastructure the auto generation of the plot for documentation has lost that parameter. The result is that now the plot does not match the default parameters in the table of the same docs.

Change History (4)

comment:1 Changed 6 years ago by richardh

This is of course only seen if a new plot has to be generated following changes to a model, hence we found it whilst sorting #1112 for the Guinier model.

comment:2 Changed 6 years ago by butler

  • Owner set to pkienzle
  • Status changed from new to assigned

comment:3 Changed 6 years ago by pkienzle

Fixed in sasmodels. Close ticket when verified.

comment:4 Changed 6 years ago by butler

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

Fixed by commit 7b9e4ddca7ad3551d8fe3696d2afe75b0018599e. Tested that it does as advertized. Was pushed directly to master rather than through a PR as was a rather simple fix and sasmodels cannot yet have builds for testing. Basically created a DEFAULT_BACKGROUND and set to 1e-3 in direct_model.py rather than set it to 1e-3 in the two places it currently is set: direct_model.py and modelinfo.py)

Now closing ticket

Note: See TracTickets for help on using tickets.