Skip to content

Normalized scales#28

Merged
guillermoaguilar merged 6 commits into
mainfrom
feat/normalized
Jun 24, 2026
Merged

Normalized scales#28
guillermoaguilar merged 6 commits into
mainfrom
feat/normalized

Conversation

@JorisVincent

@JorisVincent JorisVincent commented Jun 19, 2026

Copy link
Copy Markdown
Collaborator

Implement normalization of scales, and of confidence intervals to range [0, 1].

Implemented in the wrangling, and an option in the overall scale estimation function.

With tests, also against the example datasets analyzed in R.

Closes #25

@JorisVincent JorisVincent self-assigned this Jun 19, 2026
@JorisVincent JorisVincent added the enhancement New feature or request label Jun 19, 2026
@JorisVincent

Copy link
Copy Markdown
Collaborator Author

Now that I'm thinking about it, is doing this an option the best way? It's very straightforward, cheap calculations -- both for normalizing the scales and the bootstrap. It doesn't require rerunning the estimation or the bootstrapping.

We could also just include both the unnormalized and the normalized scales (and CIs if calculated) in the scale_estimation output dict? What do you think, @guillermoaguilar ?

@guillermoaguilar guillermoaguilar left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it is ok as it is, usually we do want only want of those, either normalized or not. So it is ok as a parameter option.
Looks good, merging!

@guillermoaguilar
guillermoaguilar merged commit 63569da into main Jun 24, 2026
12 checks passed
@JorisVincent
JorisVincent deleted the feat/normalized branch June 24, 2026 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Normalized scales

2 participants