Update pineappl 0.8.2

[giacomomagni]

Update pineappl to version 0.8.2

[giacomomagni]

do we want to change the other occurrences of convolute in convolve before merging ?

[Radonirinaunimi]

do we want to change the other occurrences of convolute in convolve before merging ?

I'd say no. We should just change the instances that are affected by pineappl. The pineappl renaming was already too much 🙈

[felixhekhorn]

do we want to change the other occurrences of convolute in convolve before merging ?

I'd say yes 😇 because the verb is just wrong (well non of us is a native speaker ...) - so you can have the decisive vote @giacomomagni if you wish

[giacomomagni]

Yes the math operation is convolve so better fix it.

[Radonirinaunimi]

Sure, but I don't think we should get into the habits of breaking APIs just for that sake. If we do so I am pretty sure that there will be many things that we could break. Here it is completely harmless because these things are not exposed, but still...

[giacomomagni]

I totally agree, but here we are only changing internal functions and docs, so it's actually a fix.

[giacomomagni]

shall I merge?

[felixhekhorn]

Sure, but I don't think we should get into the habits of breaking APIs just for that sake. If we do so I am pretty sure that there will be many things that we could break. Here it is completely harmless because these things are not exposed, but still...

on general grounds I agree - however, on eko we are facing the opposite: there are PRs which implement new good features, but are breaking and thus get delayed forever