I was hoping to address Issue #41. It looks like this was mostly due to some off-by-one indexing errors. I added one or two simple tests to address the issue, and needed to adjust other tests which were relying on the extra, empty rows being present. All tests are currently passing locally.
There are some extra changes appearing due to fantomas formatting. I'll comment to highlight the relevant changes where made.
I definitely think that the discussion in the issue about how this could break existing processes should be addressed before this can be moved forward.
I was hoping to address Issue #41. It looks like this was mostly due to some off-by-one indexing errors. I added one or two simple tests to address the issue, and needed to adjust other tests which were relying on the extra, empty rows being present. All tests are currently passing locally.
There are some extra changes appearing due to fantomas formatting. I'll comment to highlight the relevant changes where made.
I definitely think that the discussion in the issue about how this could break existing processes should be addressed before this can be moved forward.