-
Notifications
You must be signed in to change notification settings - Fork 76
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
Wrong claims values after 24 development periods [BUG] #438
Comments
Thanks, I haven't yet run our test suite against pandas>=2.0. Sounds like there are breaking changes that need to be addressed. This issue gives me a good place to start looking. |
@andrejakobsen welcome! I've been trying to figure out what's going on, but as a temporary solution, you can do something like .dev_to_val().val_to_dev() This will at least gets you back to the correct age periods. Will continue to investigate later today... |
I was suspecting it might be related to pandas 2.0 since I haven't had any issues before. Glad to have it confirmed 👍🏻 Should I make a PR for Thanks for the tip, @kennethshsu! I will probably just stick to an earlier version of pandas for now. |
Describe the bug
While using the package on monthly data, I noticed that the second origin row was missing claims values for the last development period. It looked like one value was missing and the rest got offset. Upon closer inspection, the 25th development period gets set equal to the sum of the values at development periods 23 and 25, where 25 gets the value of 26 and 27 gets the value at 28, etc. So after 24 development periods the values are offset by 1 period. This also occurs for the first origin period but at development period 25. However, all origin periods after the second do not seem to have this issue.
To Reproduce
Steps to reproduce the behavior. Code should be self-contained and runnable against publicly available data. Here is a test file that makes it easy to see the issue: test_triangle.csv
Expected behavior
A clear and concise description of what you expected to happen. If it can be expressed in code, then that is better. The complete code can serve as a unit test.
Desktop (please complete the following information):
The text was updated successfully, but these errors were encountered: