-
Notifications
You must be signed in to change notification settings - Fork 53
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
pmvpdd differences in Py and R. #117
Comments
@marcelschweiker great now I can tag you in the issues. |
Review@marcelschweiker I looked into this issue again and I can say that changing the equation for t_cla does not affect the results and I could use either of the following:
on the other hand I need to use
changing the t_cla equation also does not affect the number of iterations needed to reach convergence. SolutionSince all my pmv_ppd test are passing, if also the tests in comf are passing we can keep the code as it is and this issue. Since we have this issue that describe the problem and document that these code differences do not affect the results. ON the other hand, if the tests in R are not passing we will need to decide which formulation we should use and inform the relative standards. Are you okay with this solution? |
Describe the bug
Her are the differences we have identified
xf
In py the function is link
xf = t_cla / 50
in R is link
xf <- xn
tcla
py
t_cla = taa + (35.5 - tdb) / (3.5 * icl + 0.1)
r
tclA <- taa + (35.5-ta) / (3.5 * (6.45 * icl + .1))
@FedericoTartarini and marcel to review
The text was updated successfully, but these errors were encountered: