Closed benmalef closed 1 month ago
MLCommons CLA bot All contributors have signed the MLCommons CLA ✍️ ✅
Attention: Patch coverage is 94.59459%
with 2 lines
in your changes missing coverage. Please review.
Project coverage is 94.41%. Comparing base (
83795aa
) to head (86e4188
).:exclamation: Current head 86e4188 differs from pull request most recent head 48abb3e
Please upload reports for the commit 48abb3e to get more accurate results.
Files | Patch % | Lines |
---|---|---|
GANDLF/models/dynunet_wrapper.py | 92.85% | 2 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@sarthakpati I made the proposed changes. Thanks for the review.
Hi @sarthakpati, I believe that it is ready for merging.
Brief description
This PR adds the dynunet model.
Proposed Changes
Important notes I have added a docker-compose file. It is used for development purposes. I will delete it when the task is completed.
Checklist
CONTRIBUTING
guide has been followed.typing
is used to provide type hints, including and not limited to usingOptional
if a variable has a pre-defined value).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].