You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Really sorry for the delay. I recently had some time to go back to this and look. I was fitting distributions using Scipy and I didn't really take the time to understand how they were fitting the distributions. The R-package and the NCAR program are using l-moments to fit the distribution.
I implemented the lmoments3 package and now the results are consistent with R and NCAR.
Thank you guys for pointing this out and I'm really sorry this took so long.
Really sorry for the delay. I recently had some time to go back to this and look. I was fitting distributions using Scipy and I didn't really take the time to understand how they were fitting the distributions. The R-package and the NCAR program are using l-moments to fit the distribution.
I implemented the lmoments3 package and now the results are consistent with R and NCAR.
Thank you guys for pointing this out and I'm really sorry this took so long.
Also, I've updated for pep8 compliance and started adding testing. Please keep looking at the code and let me know if there are any other issues.
Thanks!!
Originally posted by @e-baumer in #2 (comment)
The text was updated successfully, but these errors were encountered: