`y` instead of `fy` is a bug in the old code. rather than conditionally copying the array depending on the accessor function, we should just always copy it (because we are making changes to it -- we don't want to change the original)