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.

Sign up for GitHub

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

Jump to bottom

Clean up input validation in calc_FiniteMixture() #142

Merged

RLumSK merged 1 commit into R-Lum:dev_0.9.x from mcol:calc_FiniteMixture

Aug 14, 2024

Contributor

mcol commented Aug 14, 2024

Various cleanups found while I was looking at this function:

simplified and streamlined a bit the logic for input validation
changed the occurrences of cat(paste("error message")); stop(domain = NA) to stop("error message", call. = FALSE), which makes it easier to test and is what is more commonly used
adjusted a bit the error messages
expanded tests on inputs


          Clean up input validation in calc_FiniteMixture().

6272a7b

RLumSK merged commit dc1ea56 into R-Lum:dev_0.9.x

10 checks passed

mcol deleted the calc_FiniteMixture branch

August 15, 2024 08:37

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet