should have a copy parameter here and allow for inplace mod. Don't you think. We can add it later in another PR