-
Notifications
You must be signed in to change notification settings - Fork 739
Update tumor detection pipline with new components #697
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.
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
Conversation
Signed-off-by: Behrooz <[email protected]>
Signed-off-by: Behrooz <[email protected]>
Signed-off-by: Behrooz <[email protected]>
Signed-off-by: Behrooz <[email protected]>
Hi @drbeh , Do you also need to update the code under Thanks. |
Hey @Nic-Ma, yes of course. All the tumor detection tutorials need to be updated. |
Signed-off-by: Behrooz <[email protected]>
Signed-off-by: Behrooz <[email protected]>
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
Signed-off-by: Behrooz <[email protected]>
Signed-off-by: Behrooz <[email protected]>
Signed-off-by: Behrooz <[email protected]>
Signed-off-by: Behrooz <[email protected]>
Signed-off-by: Behrooz <[email protected]>
Signed-off-by: Behrooz <[email protected]>
Signed-off-by: Behrooz <[email protected]>
Signed-off-by: Behrooz <[email protected]>
Signed-off-by: Behrooz <[email protected]>
Signed-off-by: Behrooz <[email protected]>
Signed-off-by: Behrooz <[email protected]>
Signed-off-by: Behrooz <[email protected]>
Signed-off-by: Behrooz <[email protected]>
Signed-off-by: Behrooz <[email protected]>
@Nic-Ma this pipeline is already tested by @ZiyueXu77 and FROC is maintained (~0.7). |
Hi @drbeh , Could you please help remove the 2 CSV files from the PR? It makes the PR around 200000 lines change. Thanks. |
Signed-off-by: Behrooz <[email protected]>
Hi @Nic-Ma, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the update, looks good to me.
@wyli Do you have any other comments?
Thanks.
* Update tumor detection pipline with new components Signed-off-by: Behrooz <[email protected]> * formatting Signed-off-by: Behrooz <[email protected]> * formatting Signed-off-by: Behrooz <[email protected]> * Add sub dataset Signed-off-by: Behrooz <[email protected]> * Convert xxxD transforms to xxxd Signed-off-by: Behrooz <[email protected]> * Update pipline with nvtx annotation Signed-off-by: Behrooz <[email protected]> * Add transforms to validation Signed-off-by: Behrooz <[email protected]> * Update tumor notebook Signed-off-by: Behrooz <[email protected]> * Update torch-based pipeline Signed-off-by: Behrooz <[email protected]> * Update perfomance profiling code Signed-off-by: Behrooz <[email protected]> * Update grid shape Signed-off-by: Behrooz <[email protected]> * Update Signed-off-by: Behrooz <[email protected]> * Update a comment Signed-off-by: Behrooz <[email protected]> * Update training/validation dataset and their links Signed-off-by: Behrooz <[email protected]> * Fix image path Signed-off-by: Behrooz <[email protected]> * Change to validate by default Signed-off-by: Behrooz <[email protected]> * Flip back the location dims Signed-off-by: Behrooz <[email protected]> * Add filtered datasets Signed-off-by: Behrooz <[email protected]> * Update csv loading Signed-off-by: Behrooz <[email protected]> * Update validation defaults Signed-off-by: Behrooz <[email protected]> * Remove csv files Signed-off-by: Behrooz <[email protected]> Co-authored-by: Nic Ma <[email protected]>
Fixes #636
Description
Updates the ignite-based tumor detection pipeline that uses numpy and pytorch tensors with the new components in MONAI.
Status
Ready
Checks
./runner [-p <regex_pattern>]