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

Add logoddsratio-functions #569

Merged
merged 10 commits into from
Mar 10, 2023
Merged

Add logoddsratio-functions #569

merged 10 commits into from
Mar 10, 2023

Conversation

strengejacke
Copy link
Member

See #568
Adds shortcuts for oddsratio_to*(log = TRUE) resp. *_to_oddsratio(log = TRUE).

@strengejacke
Copy link
Member Author

Since we explicitly have functions for the logOR here, I thought to omit the log argument, but @mattansb you had it in other functions already, so I followed the current implementation and also added log = TRUE as argument.

@strengejacke strengejacke requested a review from mattansb March 6, 2023 11:09
@mattansb
Copy link
Member

mattansb commented Mar 9, 2023

@strengejacke I think this can be merged. I will address the other points in #568 in a new PR.

[skip ci]
@mattansb mattansb merged commit 66341ee into main Mar 10, 2023
@mattansb mattansb deleted the logoddsratio branch March 10, 2023 08:14
@strengejacke
Copy link
Member Author

Thanks!

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.

2 participants