Skip to content

Step Yeo Johnson contains an unused line #1464

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

Closed
dshemetov opened this issue Mar 31, 2025 · 1 comment · Fixed by #1465
Closed

Step Yeo Johnson contains an unused line #1464

dshemetov opened this issue Mar 31, 2025 · 1 comment · Fixed by #1465

Comments

@dshemetov
Copy link
Contributor

Hi, we're reusing some of your step_YeoJohnson code in epipredict and ran across an unused line of code in the YJ parameter estimation here. We looked into the original Yeo Johnson (2000) paper and found that indeed, mu_t is not needed for the estimation process, so we removed the line in our extension without issue. Figured we'd upstream the change.

Copy link

This issue has been automatically locked. If you believe you have found a related problem, please file a new issue (with a reprex https://reprex.tidyverse.org) and link to this issue.

@github-actions github-actions bot locked and limited conversation to collaborators Apr 16, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant