Added functionality related to DP training

scap3yvt commented 2 months ago

Fixes #944

Proposed Changes

Add DP-enabled training and inference from private repo
Most of the code is copy-paste from the private repo, with some added documentation and changes from raised errors to assertions.

[x] CONTRIBUTING guide has been followed.
[x] PR is based on the current GaNDLF master .
[x] Non-breaking change (does not break existing functionality): provide as many details as possible for any breaking change.
[x] 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).
[x] Code has been blacked for style consistency and linting.
[x] If applicable, version information has been updated in GANDLF/version.py.
[x] If adding a git submodule, add to list of exceptions for black styling in pyproject.toml file.
[x] Usage documentation has been updated, if appropriate.
[x] Tests added or modified to cover the changes; if coverage is reduced, please give explanation.
[x] 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].
[x] The logging library is being used and no print statements are left.

github-actions[bot] commented 2 months ago

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

sarthakpati commented 2 months ago

This seems to be okay for me. Any comments, @VukW and @brandon-edwards?

brandon-edwards commented 2 months ago

This looks reasonable to me as well, but I have not tested it.

sarthakpati commented 2 months ago

This should not be merged until #947 is merged and the master gets tagged.

sarthakpati commented 2 months ago

Support ticket generated with Codacy to explore the coverage issue.