Codecov not correctly merging reports

Support
21

Hi @josh146, apologies on the wait here. I think what is happening is that we are getting conflicts for __init__.py in the first 5 reports, which is why you didn’t see it in your screenshot. qchem does actually run things in pennylane it seems as a site-package, which is why you are seeing coverage for it.

I would suggest a pathfix of

  - "/home/runner/work/pennylane/pennylane/pennylane/::pennylane/"

Would you be able to see if that helps?

22

Thanks Tom! I’m testing that now here: Path fix for codecov by josh146 · Pull Request #832 · PennyLaneAI/pennylane · GitHub

23

Unfortunately, it didn’t seem to work :frowning: Is the left hand side of the pathfix the location to site-packages?

24

Hi @josh146, apologies on the delay here. I wasn’t able to find an elegant solution to this issue. Instead, I wrote up a hack, which I think will solve your issue. Would you mind trying this out?

25

Thanks Tom, this works!

I really appreciate your help to fix this issue. Out of curiosity, is it a bug causing the underlying issue, or something to do with out configuration?

1 Like
26

Hi @josh146, this is a limitation of our current configuration options. I’ll be looking into how to fix this in the future, but I really appreciate your patience here.

27

No worries, thanks Tom!

1 Like