codecov-commenter
commented
1 year ago Codecov Report
Patch coverage: 100.00% and project coverage change: -1.64 :warning:
Comparison is base (
a886f74) 72.22% compared to head (7b9efb4) 70.58%.
Additional details and impacted files
```diff @@ Coverage Diff @@ ## main #37 +/- ## ========================================== - Coverage 72.22% 70.58% -1.64% ========================================== Files 7 7 Lines 468 442 -26 ========================================== - Hits 338 312 -26 Misses 130 130 ``` | Flag | Coverage Δ | | |---|---|---| | unittests | `70.58% <100.00%> (-1.64%)` | :arrow_down: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=qiboteam#carryforward-flags-in-the-pull-request-comment) to find out more. | [Impacted Files](https://app.codecov.io/gh/qiboteam/qibosoq/pull/37?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=qiboteam) | Coverage Δ | | |---|---|---| | [src/qibosoq/components.py](https://app.codecov.io/gh/qiboteam/qibosoq/pull/37?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=qiboteam#diff-c3JjL3FpYm9zb3EvY29tcG9uZW50cy5weQ==) | `100.00% <100.00%> (ø)` | | | [src/qibosoq/qick\_programs.py](https://app.codecov.io/gh/qiboteam/qibosoq/pull/37?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=qiboteam#diff-c3JjL3FpYm9zb3EvcWlja19wcm9ncmFtcy5weQ==) | `86.50% <100.00%> (ø)` | | ... and [1 file with indirect coverage changes](https://app.codecov.io/gh/qiboteam/qibosoq/pull/37/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=qiboteam):umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
I added soft_avgs for raw acquisition, but if it's defined as None then will throw errors. There is no need of having the parameter there in any case, I will simply add it to the dictionary if needed.
Checklist before merge: