Closed andrejakobsen closed 1 year ago
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...
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.
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 requirements.txt
to have pandas>=0.23,<2.0.0
for the time being?
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):