Skip to content

Conversation

@Remi-Gau
Copy link
Contributor

@codecov
Copy link

codecov bot commented Dec 24, 2020

Codecov Report

Merging #237 (3b6bbe5) into dev (d15cc55) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##              dev     #237   +/-   ##
=======================================
  Coverage   46.29%   46.29%           
=======================================
  Files          98       98           
  Lines        1754     1754           
=======================================
  Hits          812      812           
  Misses        942      942           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d15cc55...32c7283. Read the comment docs.

@Remi-Gau
Copy link
Contributor Author

refactor getPrefix and functionalQA

In src/getPrefix.m, line 3
| function [prefix, motionRegressorPrefix] = getPrefix(step, opt, funcFWHM)
|                                            ^^^^^^^^^ metric: exceeded cyc: measured 17 > limit 12
In src/QA/functionalQA.m, line 3
| function functionalQA(opt)
|          ^^^^^^^^^^^^ metric: exceeded cnest: measured 5 > limit 4
MISS_HIT Metric Summary: 161 file(s) analysed, 2 metric deviations(s)
Error: Process completed with exit code 1.

@Remi-Gau Remi-Gau merged commit 9783fc4 into cpp-lln-lab:dev Dec 24, 2020
@Remi-Gau Remi-Gau deleted the remi-update_miss_hit branch December 24, 2020 11:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[INFRA] in CI make sure that MISS_HIT style runs before MISS_HIT metric

1 participant