Skip to content

Conversation

@pkienzle
Copy link
Contributor

Fixes #606

Check by running the bumps GUI with model example/fit.py:

python -m bumps.cli example/fit.py core_shell_cylinder --edit

Toggling length_pd and theta_pd fitting should give the limits as the parameter defaults.

Note: the --edit option only works if you have wx in your python environment.

@pkienzle pkienzle changed the title See pd limits to match sasview Set pd limits to match sasview Jan 21, 2025
@krzywon krzywon changed the base branch from master to release_1.0.9 January 22, 2025 19:41
@krzywon
Copy link
Collaborator

krzywon commented Jan 28, 2025

Lower priority for the next release

@krzywon krzywon changed the base branch from release_1.0.9 to master February 25, 2025 16:54
@butlerpd butlerpd requested a review from krzywon April 22, 2025 13:48
@krzywon krzywon merged commit 7276d57 into master May 6, 2025
18 checks passed
@krzywon krzywon deleted the 606_pd_limits branch May 6, 2025 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Polydispersity limits is incorrectly set in bumps_model.py

5 participants