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

Phi and theta stuck at zero in 2D theory calculations #3053

Closed
butlerpd opened this issue Aug 17, 2024 · 3 comments · Fixed by #3100
Closed

Phi and theta stuck at zero in 2D theory calculations #3053

butlerpd opened this issue Aug 17, 2024 · 3 comments · Fixed by #3100
Labels
Critical High priority

Comments

@butlerpd
Copy link
Member

Describe the bug
When plotting oriented 2D scattering patterns from theory models, setting either theta or phi to 0 at any point, disables that parameter. Any additional changes to those angles are not updated in the simulated data images. This makes it difficult to manually fit. It is not clear (and hard to test) if the same would happen if the fitting algorithm sets one of those values to zero which would be extremely problematic.

To Reproduce
Steps to reproduce the behavior:

  1. create a theory model for any anisotropic shape (e.g. cylinder) and click the 2D box before plotting
  2. change angles theta and phi (and psi for prism models)
  3. Now set one to zero
  4. Try to move off of zero
  5. Pattern does not change from that at 0

Expected behavior
the plot (and underlying theory data) should recalculate with the new angel and not use zero as being unchangeble.

SasView version (please complete the following information):

  • Version: 6.0.0b2

Operating system (please complete the following information):

  • OS: Windows 10

Additional context
Identified during the NIST CNR summer school and thus labelled 6.0.0 release as agreed - However, whether this is a longer standing issue has yet to checked

@butlerpd butlerpd added Critical High priority SasView 6.0.0 Required for 6.0.0 release labels Aug 17, 2024
@krzywon
Copy link
Contributor

krzywon commented Aug 19, 2024

This issue is present in v5.0.6. In order to prevent scope creep, this should not be fixed for v6.0.0.

@wpotrzebowski wpotrzebowski removed the SasView 6.0.0 Required for 6.0.0 release label Aug 20, 2024
@davidnwobi
Copy link

This also happens for other parameters besides the angle. I've seen the same behaviour with the parameters of the cylinder and ellipsoid models in the 1D case. It seems to be a more general issue with all the parameters; in this case: sld, sld_solvent, radius and solvent.

@krzywon krzywon linked a pull request Sep 6, 2024 that will close this issue
7 tasks
@smk78
Copy link
Contributor

smk78 commented Sep 20, 2024

Retested using https://github.com/SasView/sasview/actions/runs/10638002500 on W10/x64.
The issue described above appears to be fixed and the 2D pattern changes as you wouuld expect, even after setting one or other of theta or phi to 0.

@smk78 smk78 closed this as completed Sep 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Critical High priority
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants