Opened 8 years ago

Last modified 5 years ago

#473 new defect

Multiplicity models do not appear in drop-down in Custom Model > Sum|Multi

Reported by: smk78 Owned by:
Priority: major Milestone: SasView 5.1.0
Component: SasView Keywords:
Cc: Work Package: SasModels Infrastructure

Description (last modified by butler)

Reported by Saeed Mohan (saeed.mohan@…):

I was attempting to add in my own custom model to Sasview, which is based on the UnifiedPowerRg model. I thought the easiest thing to do would be to use the Sum Multi|(p1, p2) function under Fitting > Edit Custom Model, however I am not able to find this model in the options list.

Checked by SMK. He is correct.

What is more, the same bug exists in v3.0.0 and v2.2.1!!!

PDB checked and it appears that no multiplicity models appear in the dropdown (eg. RPA10, onion, etc). This is probably working as designed since I suspect the interface was not obvious for a multiplicity model? Interestingly there is also a very strange behavior that the shape independent → Absolute_Power_Law seems to have been renamed in this dropdown to PowerLawAbsModel … perhaps due to the space in the name?

Change History (6)

comment:1 Changed 8 years ago by butler

  • Description modified (diff)
  • Priority changed from critical to major
  • Summary changed from UnifiedPowerRg model does not appear in drop-down in Custom Model > Sum|Multi to Multiplicity models do not appear in drop-down in Custom Model > Sum|Multi

comment:2 Changed 8 years ago by butler

  • Priority changed from major to critical
  • Work Package changed from SasView Bug Fixing to SasModels Redesign

The way this is currently handled is in ModelManagerBase in models.py. every model is explicitly added to a variety of lists and dictionaries. The sum model dropdown uses the model_name_list to populate itself. ModelManagerBase has two mutually exclusive lists: model_name_list and multi_func_list. This last is the list of all models which have a multiplicity parameter (selectable number of levels). One CAN force these to also be members of model_name_list but the sum model editor will then only produce a sum model with 1 level as there is currently no way to select that. This would require a GUI change in src.sas.perspectives.calculator.model_editor.TextDialog to handle.

We will have to deal with this as part of sasmodels.

comment:3 Changed 8 years ago by butler

  • Description modified (diff)

with regard to the naming issue it appears that the sum model editor uses the file name (PowerLawAbsModel.py) while the fit panel's drop down uses the self.name parameter of the model. These seem to usually be the same or similar except in the PowerLawAbsModel which is alphebetically far away from the otherwise first Absolute Power_Law.

The GUI should be consistent and always use one or the other. This should be addressed as part of the new sasmodels implementation as well though it may be appropriate to divide these into two tickets

comment:4 Changed 8 years ago by butler

  • Milestone changed from SasView 4.0.0 to SasView Next Release +1
  • Priority changed from critical to major

naming should be consistant as of 4.0, but multiplicity models will not be able to sum/multiply for this release. Question of whether that is something we do.

comment:5 Changed 6 years ago by butler

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

comment:6 Changed 5 years ago by butler

  • Work Package changed from SasModels Redesign to SasModels Infrastructure
Note: See TracTickets for help on using tickets.