A potential typo/bug in averaging s_test?

[zlin7]

In calc_s_test_single, it seems like the intended operation is to add all the sampled s_test_i together and divide the sum by r. However, it is actually extending the list. Because of this, it seems like when I set r>1, calc_influence_single just completely skips s_test_vec[len(grad_z_vec):]. Am I not understanding what this is doing correctly, or is this a typo? I'm quite unsure because this looks like something intentional, as it's explicitly converted to a list of tensors.

[fjzpcmj]

I think it is a bug. The code only uses the first s_test to calculate influence even when r>1.

[mkirchhof]

Just verified and you are right. Gradients are appended instead of added. Fixed this in my repo in this commit a959de6

I can create a pull request for if @nimarb would like to merge it.