-
Notifications
You must be signed in to change notification settings - Fork 29
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
qs.reports.html compounded=False not accurate for Calmer / ROMaD / RaR #29
Comments
Why do you think that compounded=True is incorrect?
Robert Grzesik 347-635-3416
…On Mon, Apr 22, 2024 at 4:36 PM Koha101 ***@***.***> wrote:
When calling qs.reports.html with the compounded=False argument, the
Calmer and ROMaD are calling the CAGR function without passing the
compounded argument. This defaults CAGR to calculate with compounded=True
for the two calculations, overestimating the Calmer and ROMaD.
I don't use RaR, but it seems like compounded is not passed to CAGR for it
as well.
—
Reply to this email directly, view it on GitHub
<#29>, or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAIYQK4CNFJUVKH2UIQARXDY6VYFNAVCNFSM6AAAAABGTPDXGOVHI2DSMVQWIX3LMV43ASLTON2WKOZSGI2TONBRHEZTGNA>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
When calling qs.reports.html with the compounded=False argument, the Calmer and ROMaD are calling the CAGR function without passing the compounded argument. This defaults CAGR to calculate with compounded=True for the two calculations, overestimating the Calmer and ROMaD.
I don't use RaR, but it seems like compounded is not passed to CAGR for it as well.
The text was updated successfully, but these errors were encountered: