Fix code issues reported by codacy

[benmalef]

Fixes #800

Proposed Changes

• Fix some code issues based on codacy. https://app.codacy.com/gh/mlcommons/GaNDLF/issues/current?levels=Warning&categories=CodeStyle

Checklist

• CONTRIBUTING guide has been followed.
• PR is based on 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 (ensure typing is used to provide type hints, including and not limited to using Optional if a variable has a pre-defined value).
• Code has been blacked for style consistency and linting.
• 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

All modified and coverable lines are covered by tests ✅

Project coverage is 95.09%. Comparing base (b9557f6) to head (df9469e).

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #808      +/-   ##
==========================================
- Coverage   95.10%   95.09%   -0.01%     
==========================================
  Files         122      122              
  Lines        8347     8344       -3     
==========================================
- Hits         7938     7935       -3     
  Misses        409      409              

Flag	Coverage Δ
unittests	95.09% <100.00%> (-0.01%)	⬇️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[sarthakpati]

LGTM pending CLA: https://github.com/mlcommons/GaNDLF/blob/master/CONTRIBUTING.md#:~:text=All%20parties%20that%20have%20commits%20to%20a%20PR%20should%20sign%20the%20MLCommons%20CLA

[VukW]

pytest CI workfolw fails beause of new version of dicom-anonymizer just was published (1.0.11 -> 1.0.12) and it's not a 100% backward compatible.
Fix: #836

[sarthakpati]

pytest CI workfolw fails beause of new version of dicom-anonymizer just was published (1.0.11 -> 1.0.12) and it's not a 100% backward compatible. Fix: #836

Thanks!

I am guessing this PR also needs to be re-linted (after #832).

[benmalef]

Can you check it? I have signed the MLCommons CLA.

[sarthakpati]

Ah okay, seems like the CLA is now passing. There are issues with the other integration tests (unit tests and linting) which are preventing the merge.

[VukW]

LGTM 👍