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

Histoparams #37

Merged
merged 24 commits into from
Mar 23, 2023
Merged

Histoparams #37

merged 24 commits into from
Mar 23, 2023

Conversation

mpatrou
Copy link
Collaborator

@mpatrou mpatrou commented Mar 22, 2023

Histogram Parameters.
Updates in Views/ and Models/ for the histogram parameters tab. Some functionality is taken from mantid --> DimensionSelectorWidget.py. Tests are included, too.

…on value and colinear check among them; code from mantid --> DimensionSelectorWidget.py
…ground colors set to red and fields are cleared for step that are not allowed to have values. Step values can be positive double numbers with a standard notation using built-in validators.
…lue as the selected one is updated with the selected ones previous value to ensure uniqueness. min and max values are checked to be float numbers, both (not) exist and min less than max
…am button click-submission a dictioary with all the histogram parameters is created
…onay creation in histogram button and projections values
…wn added and valid states for projections and min-max value pairs
…e mandit script, tests added, steps mandatory based on radio buttons with their tests
@rosswhitfield rosswhitfield self-requested a review March 22, 2023 14:11
src/shiver/views/histogram_parameters.py Outdated Show resolved Hide resolved
src/shiver/models/histogram.py Outdated Show resolved Hide resolved
@rosswhitfield rosswhitfield merged commit 3a53868 into next Mar 23, 2023
@rosswhitfield rosswhitfield deleted the histoparams branch March 23, 2023 13:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants