Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversion routines for reparameterized models not used in 5.x #1902

Open
krzywon opened this issue Aug 30, 2021 · 4 comments
Open

Conversion routines for reparameterized models not used in 5.x #1902

krzywon opened this issue Aug 30, 2021 · 4 comments
Labels
Blocker Prevents a different issue from being resolved Good First Issue Issues that are appropriate for newbies Housekeeping Tidying, renaming, formatting, minor refactoring, boring stuff in general
Milestone

Comments

@krzywon
Copy link
Contributor

krzywon commented Aug 30, 2021

Models that have their model parameters changed in 5.x are not run through the model conversion routines found in sasmodels/convert.py. This is causing a problem with sasmodels PR #458. This conversion is implemented in v4.x master: sasview/pagestate.py.

The conversion methods are available in the main branch but will need to be called while loading a fit project. A good place to do this might be in FittingWidget.updateParamsWithParameters().

@krzywon krzywon added the Good First Issue Issues that are appropriate for newbies label Aug 30, 2021
@butlerpd butlerpd added this to the SasView 5.0.5 milestone Sep 4, 2021
@butlerpd butlerpd added the Blocker Prevents a different issue from being resolved label Sep 4, 2021
@butlerpd
Copy link
Member

A related problem just discovered while testing sasmodels #458 is that the project loader no longer loads 4.x projects at all! Instead it reports the error:

19:43:45 - WARNING: ER(...) function ignored. Using radius_effective(mode, ...) instead if it exists.
19:43:45 - ERROR: Error while reading the project file: No module named 'sas.sascalc.dataloader.readers.cansas_reader_HDF5'
19:43:45 - ERROR: Error while converting the project file: local variable 'datasets' referenced before assignment

@krzywon
Copy link
Contributor Author

krzywon commented Oct 15, 2021

@butlerpd Could you attach a project that is failing for you? I've attached a simple sphere fit that I generated using v4.2.2 that loads when running from the sasview main and sasmodels master branches.
save_v4_2_2_sphere.zip

Edit: I tested loading this project using both the BroadPeak model reparameterization branch as well as master and both branches are successful in loading it. Perhaps this is an issue with the broad peak?

@butlerpd
Copy link
Member

did you try this with the installer version of main? it may be that there is a dependency issue?

@butlerpd
Copy link
Member

butlerpd commented Mar 1, 2022

This needs to be addressed before the next release or broad peak model may now break things sasmodels#458

@lucas-wilkins lucas-wilkins added the Housekeeping Tidying, renaming, formatting, minor refactoring, boring stuff in general label Sep 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Blocker Prevents a different issue from being resolved Good First Issue Issues that are appropriate for newbies Housekeeping Tidying, renaming, formatting, minor refactoring, boring stuff in general
Projects
None yet
Development

No branches or pull requests

3 participants