`return_copy` should probably be dropped from `kwargs` when checking cache hits

[kushalkolar]

@clewis7 posting this because it just popped into my head, don't worry about it until you're back

the return_copy kwarg is irrelevant for determine a cache hit/miss therefore it should be dropped for the comparison.