8000 First level of refactoring by sarthakpati · Pull Request #70 · mlcommons/GaNDLF · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

First level of refactoring #70

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 337 commits into from
Jun 11, 2021
Merged

First level of refactoring #70

merged 337 commits into from
Jun 11, 2021

Conversation

sarthakpati
Copy link
Collaborator

Fixes #40

Proposed Changes

  • refactor of codebase
  • new features are included in HISTORY

Checklist

  • I have read the CONTRIBUTING guide
  • My PR is based from the current GANDLF master
  • Non-breaking change (would not break existing functionality): if changes breaks current code, please provide as many details as possible
  • Function/class source code documentation added/updated
  • Usage documentation has been updated, if appropriate
  • History has been updated, if appropriate
  • Tests added or modified to cover the changes
  • Integration tests passed locally by running pytest

sarthakpati and others added 30 commits March 27, 2021 07:56
@sarthakpati
GANDLF --> GaNDLF in documentation and text
1431ddf
@sarthakpati
msdnet fixed
57e39d3
@sarthakpati
amp should be disabled for msdnet
915d783
@sarthakpati
msdnet added to tests
42ed3f8
@Geeks-Sid
Merge pull request #30 from sarthakpati/refactor
1845f36
@sarthakpati
added coverage config
8e57d49
@sarthakpati
Merge branch 'refactor' of https://github.com/CBICA/GaNDLF into refactor
b1e5b54
@sarthakpati
added a vebose option
8000 34716aa
@sarthakpati
printing memory stats
067ce48
@sarthakpati
fixed #31
4669e3c
@sarthakpati
added coverage
8296e89
@sarthakpati
added instructions to obtain code coverage for tests
ef16287
@sarthakpati
using scheduler
3fbf53b
@sarthakpati
more scheduling options added
fdcabac
@sarthakpati
all options updated
636796d
@sarthakpati
scheduler test added
b8d71a8
@sarthakpati
added info about schedulers
df0c922
@sarthakpati
fixing a tensor error
be2fc56
@sarthakpati
added cpu utilization stats
6a4ef68
@sarthakpati
better parsing
1f4d4fa
@sarthakpati
correct printing
d22e0da
@sarthakpati
Merge branch 'master' of https://github.com/sarthakpati/GANDLF into r…
a5c9aad 
…efactor
@sarthakpati
Merge branch 'master' of https://github.com/sarthakpati/GANDLF into r…
bc3db0e 
…efactor
@sarthakpati
setting up for auto-resize of output
2691d7e
@sarthakpati
Merge branch 'master' of https://github.com/sarthakpati/GANDLF into r…
60d4b38 
…efactor
@Geeks-Sid
Merge pull request #33 from sarthakpati/refactor
a6eb610
@sarthakpati
num_classes fix for regression/classification from master #32
6a2eaaf
@sarthakpati
segmentation fix
fada20a
@sarthakpati
Merge branch 'master' of https://github.com/CBICA/GaNDLF into refactor
0e110a6
@Geeks-Sid
Merge pull request #35 from sarthakpati/refactor
4bb9f87
sarthakpati and others added 25 commits June 9, 2021 23:02
@sarthakpati
removed "headers" from inference
85d2fae
@sarthakpati
removed "headers" from training
96471a4
@sarthakpati
explicit "headers" removed
c688f45
@sarthakpati
Merge branch 'refactor_63' of https://github.com/sarthakpati/GANDLF i…
02186fc 
…nto refactor_inference_consolidate
@sarthakpati
consolidated code
a81067d
@Geeks-Sid
Merge pull request #64 from sarthakpati/refactor_63
50133f9
@sarthakpati
Merge branch 'refactor' of https://github.com/CBICA/GaNDLF into refac…
cfe12b5 
…tor_inference_consolidate
@sarthakpati
ensuring a single model is always saved
d3ff5dc
@sarthakpati
Merge branch 'refactor' of https://github.com/sarthakpati/GANDLF into…
9a30a41 
… refactor_inference_consolidate
@Geeks-Sid
Merge pull request #65 from sarthakpati/refactor
2dff291 
Ensuring a single model is always saved
@sarthakpati
added check
1261c64
@sarthakpati
Merge pull request #66 from sarthakpati/refactor_53
c666a59 
Added minimum patch size check for unets
@sarthakpati
minor doc updates
e6e3eff
@Geeks-Sid
Merge pull request #67 from sarthakpati/refactor
e142d0b
@Geeks-Sid
histopath inference is now
8e10a81
@Geeks-Sid
Merge branch 'refactor' of https://github.com/CBICA/GaNDLF into refactor
da1de67
@sarthakpati
Merge branch 'refactor' of https://github.com/CBICA/GaNDLF into refac…
219782e 
…tor_inference_consolidate
@sarthakpati
unnecessary imports removed
ef4b021
@Geeks-Sid
Merge pull request #68 from sarthakpati/refactor_inference_consolidate
e41ca29
@sarthakpati
using num_works from the parameters
13e9e1c
@sarthakpati
generic location for processing, rather than cuda
a89b955
@sarthakpati
consolidating defaults in a single dict
392ea55
@sarthakpati
comments added
70d06c6
@sarthakpati
comment added
b2f3c0e
@sarthakpati
Merge pull request #69 from sarthakpati/refactor
cc245cd 
Refactor: minor code cleanup
@sarthakpati sarthakpati requested a review from Geeks-Sid June 11, 2021 16:13
@Geeks-Sid Geeks-Sid merged commit 1e8ae12 into master Jun 11, 2021
@Geeks-Sid Geeks-Sid deleted the refactor branch June 11, 2021 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Geeks-Sid Geeks-Sid Geeks-Sid approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Code re-organization
2 participants
@sarthakpati @Geeks-Sid
0