The STATISTIC parameter in imtime_ops::convolve

[Hertz4]

In the method imtime_ops::convolve, there is a parameter STATISITIC, for which you can pass in Fermion and Boson. And there are comments here and here saying:

// TODO: implement bosonic case and remove

Note that the two functions being convolved does not have to be simultaneously Bosonic or Fermionic, so having only one STATISTIC parameter bears its limitation.

However, more importantly, the current implementation seems valid universally regardless of the statistics of both functions.

If the developers could kindly confirm this, I believe there is an easy fix with minimal workload, which is to get rid of the STATISTIC argument, and also get rid of the not-implemented error (here and here) for Bosonic functions.

Best,
Z

[jasonkaye]

Thanks for pointing this out. I have opened PR #17 to address this issue.

[jasonkaye]

PR #17 fixing this issue now merged.