Added parameter to toggle NCC computation

[FelixSteinbauer]

Introduced a parameter (compute_ncc, defaut=True) that allows to turn off the (time-consuming) computation of the 4 NCC metrics.

Checklist

• I have read the CONTRIBUTING guide.
• My PR is based from the current GaNDLF master .
• Non-breaking change (does not break existing functionality): provide as many details as possible for any breaking change.
• Function/class source code documentation added/updated.
• Code has been blacked for style consistency.
• If applicable, version information has been updated in GANDLF/version.py.
• If adding a git submodule, add to list of exceptions for black styling in pyproject.toml file.
• Usage documentation has been updated, if appropriate.
• Tests added or modified to cover the changes; if coverage is reduced, please give explanation.
• If customized dependency installation is required (i.e., a separate pip install step is needed for PR to be functional), please ensure it is reflected in all the files that control the CI, namely: python-test.yml, and all docker files [1,2,3].

[github-actions]

MLCommons CLA bot All contributors have signed the MLCommons CLA ✍️ ✅

[codecov]

Codecov Report

Merging #717 (bc0d3fa) into master (32fd05c) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #717   +/-   ##
=======================================
  Coverage   94.72%   94.72%           
=======================================
  Files         117      117           
  Lines        8251     8253    +2     
=======================================
+ Hits         7816     7818    +2     
  Misses        435      435           

Flag	Coverage Δ
unittests	94.72% <100.00%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed	Coverage Δ
GANDLF/cli/generate_metrics.py	100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[sarthakpati]

LGTM

[FelixSteinbauer]

@sarthakpati sorry to bother you again: I don't really know how to solve the "Dev-Container CI" error. I don't see how that is connected to to changes I made and I would not know where to start resolving this... Similarly the "Codacy Static Code Analysis", how can I make the PR pass this one?

[sarthakpati]

@sarthakpati sorry to bother you again: I don't really know how to solve the "Dev-Container CI" error. I don't see how that is connected to to changes I made and I would not know where to start resolving this... Similarly the "Codacy Static Code Analysis", how can I make the PR pass this one?

Hey, the dev container ci is indeed unrelated, so we can ignore it. The codacy will just take some time. I'll reach out to them to see what the status is. Nothing for you to do!