Skip to content
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

[Auto3DSeg] Report/log failed cases in the process of data analysis #6114

Closed
dongyang0122 opened this issue Mar 7, 2023 · 0 comments · Fixed by #6202
Closed

[Auto3DSeg] Report/log failed cases in the process of data analysis #6114

dongyang0122 opened this issue Mar 7, 2023 · 0 comments · Fixed by #6202
Assignees
@mingxin-zheng
Labels
enhancement New feature or request

Comments

@dongyang0122
Copy link
Collaborator

Is your feature request related to a problem? Please describe.
The data analysis module in Auto3DSeg does not print or log information when some data is corrupted. Users may not be able to determine which data points are corrupted.

Describe the solution you'd like
We need to print/log error information using try/except methods or alternatives.
@wyli wyli added the enhancement New feature or request label Mar 9, 2023
@wyli wyli mentioned this issue Apr 11, 2023
Merged
7 tasks
wyli pushed a commit that referenced this issue Apr 11, 2023
@heyufan1995
Adding support for Multi-GPU data analyzer #6182 (#6202)
6a7f35b 
Fixes #6182  .
fixes #6114 
Added multi-gpu support for data analyzer.
Tested on NGC using 4 16g V100 for total segmentator
Speed up:
4 GPU: data analysis 6.90 mins (9.2 mins including yaml export)
1 GPU: data analysis 22.98 mins (25.33 mins including yaml export)


### Types of changes
<!--- Put an `x` in all the boxes that apply, and remove the not
applicable items -->
- [x] Non-breaking change (fix or new feature that would not break
existing functionality).
- [ ] Breaking change (fix or new feature that would cause existing
functionality to change).
- [ ] New tests added to cover the changes.
- [ ] Integration tests passed locally by running `./runtests.sh -f -u
--net --coverage`.
- [ ] Quick tests passed locally by running `./runtests.sh --quick
--unittests --disttests`.
- [ ] In-line docstrings updated.
- [ ] Documentation updated, tested `make html` command in the `docs/`
folder.

---------

Signed-off-by: heyufan1995 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mingxin-zheng mingxin-zheng

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Adding support for Multi-GPU data analyzer #6182 heyufan1995/MONAI
3 participants
@wyli @dongyang0122 @mingxin-zheng