xKDR/TSFrames.jl

Do we use `deepcopy()` when `copycols=true` in the constructor

Opened this issue · 1 comments

PR #142 adds a keyword argument for the user to specify whether to copy the columns or not. Since the package does not have any in-place modification methods is deepcopy really needed at this moment?

The package doesn't do anything in place, but the user might - the reason I included this was so that if I want to mutate my dataframe afterwards, I can ensure that that mutation doesn't affect the TSFrame by passing copycols = true.

It's a point for discussion.
My understanding is that DataFrames does the same thing, so there is an argument for mirroring its behaviour in this case.