Version 9 (modified by butler, 9 years ago) (diff)

SasView 3.1.0 Release Punch List

This is a running list of things to address for release 3.1

Remaining tickets

Tickets

Outstanding Questions (mostly from Richard)

  1. “report” button does nothing (nor for Steve King on build 83)

    FIXED

  1. The very nice help documentation works fine for me (apart from the previously mentioned irritating lack of “back” button on screen, though there is one if you right click, or use ALT/leftArrow; whilst ALT/rightArrow is forward.). However the section on the optimizers is not rendering equations so has $n$, $1-sigma$ etc.

    CHECK should be simply due to not connecting to CND

  1. I think, from experiment with data rather than looking at the code, the docs for the Hayter MSA structure factor needs to note that even if no salt has been specifically added to the sample, the “salt conc” may need to be non-zero to allow for counterions that have come off say a micelle as it charges up. One day I will code up a version of the Hayter MSA where the Debye length is explicitly entered.

    TODO

  1. The “help about” box does not report the exact build number, just has 1>

    KNOWN ISSUE - NEXT RELEASE #419

  1. Why are the parameter errors now appearing in red? (Which we usually keep around here for “fault” messages, so is giving me unnecessary stress.) - NOTE added by PDB: Vaguely remember adding red when something was wrong. Also limits sometimes show up prepopulated. QUESTION: are we accidentally using the new sasmodels?

    CHECK

  1. If I set up 3 data sets with constraints, say have M2.radius=M1.radius, M3.radius=M1.radius but then fix M1.radius at say 30 ang, I would expect the constraints to still work, but I get error message on hitting Fit in constraints page.
                    Traceback (most recent call last):
      File "sas\perspectives\fitting\fit_thread.pyc", line 82, in compute
      File "sas\perspectives\fitting\fit_thread.pyc", line 14, in map_apply
      File "sas\perspectives\fitting\fit_thread.pyc", line 11, in map_getattr
      File "sas\fit\BumpsFitting.pyc", line 262, in fit
      File "sas\fit\BumpsFitting.pyc", line 345, in run_bumps
      File "bumps\fitters.pyc", line 859, in stderr
      File "bumps\lsqerror.pyc", line 79, in hessian
      File "bumps\numdifftools.pyc", line 1159, in __call__
      File "bumps\numdifftools.pyc", line 1170, in hessian
      File "bumps\numdifftools.pyc", line 1084, in hessdiag
      File "bumps\numdifftools.pyc", line 720, in _partial_der
      File "bumps\numdifftools.pyc", line 424, in _derivative
      File "bumps\numdifftools.pyc", line 686, in _romb_extrap
    ValueError: operands could not be broadcast together with shapes (24,) (0,)
     
     None
    
    OK the work around for this is to also fix M2.radius etc manually. However if I do that and then hit Fit on constraints page, nothing happens not even an error message. So if I “Remove” the constraints on M2.radius and M3.radius, “Fit” now attempts to work but comes back with same error message as above in red, presumably because it now has an issue with radius.width which is still constrained and fitting. Indeed if I remove the constraints on radius.width the fit now works, but with independent values for the polydispersity.

I would expect the constraints to work regardless of whether the base parameters in them are fixed or adjusting.

Meanwhile perhaps there can be a quick fix in this release to put up a message that “constrained parameters must all be adjusting” ?

If anyone wants a simple 3 contrast core/shell microemulsion project, with real SANS data, to test the above just ask. I am grateful that “Save project” does now actually save all 3 data sets and models. (It would be nice if it also remembered which parameters are on/off, and in the +1 release, saved the constraints also, so that a fully working project could be emailed to someone else, as setting up the constraints etc is still non-trivial.)

{{font color|red|CHECK}}

  1. In Model Documentation, the Table of contents on left side has repeats of sections 2.2, 2.3 and 2.4. (There is also a note at top of screen about proof reading, which is no doubt still true!)

    FIXED

  1. The customised functions section of this section could do with a pointer to more docs on custom models hiding in User documentation/fitting perspective/fitting data/adding your own models

    FIXED

  1. Might be useful if the “fit option” type was display on screen somewhere.

    TICKET FOR NEXT RELEASE #434

  1. though there may be some good underlying reason, can we do away with the top level menu of the documentation? Model documentation is duplicated in User documentation, so just put Developer documentation at the bottom of the main menu and save clicks for everyone except developers (who get one more). NOTE added by PDB: this was discussed briefly at code camp. My personal feeling was similar (go directly to user docs) but would argue that developer docs do not need to be included in the GUI application. They should be available from command line or from a web page (do we push the docs yet to the web page?).

    CHECK

  1. A “compute” button on the “const & simul fit” tab for a forward calculation only but with all the constraints applied would be useful.

    TICKET FOR NEXT RELEASE

  1. In the DREAM parameter 2d correlation plots are very good but don’t actually quantify how large the correlation is. It might be useful to have some scale markers on the axes. The 2d plots are of course very small, so normal axis labels would likely look a mess, perhaps try sticks on the sides at 0 and say +-3sigma. (Normal correlation coefficients as I recall say how many standard deviations parameter A moves if parameter B moves one standard deviation.)

    CHECK