Skip to content

Update patch merging with a for loop#4793

Merged
wyli merged 3 commits intoProject-MONAI:devfrom
function2-llx:4753-improve-patch-merging
Jul 29, 2022
Merged

Update patch merging with a for loop#4793
wyli merged 3 commits intoProject-MONAI:devfrom
function2-llx:4753-improve-patch-merging

Conversation

@function2-llx
Copy link
Copy Markdown
Contributor

@function2-llx function2-llx commented Jul 29, 2022

Fixes #4753

Description

Update the patch merging process with a for loop to make it less error-prone and more readable.

Status

Ready

Types of changes

  • Non-breaking change (fix or new feature that would not break existing functionality).
  • Quick tests passed locally by running ./runtests.sh --quick --unittests --disttests.

@function2-llx function2-llx force-pushed the 4753-improve-patch-merging branch from bb67048 to c70ef06 Compare July 29, 2022 12:45
@function2-llx function2-llx force-pushed the 4753-improve-patch-merging branch from c70ef06 to 8a4ec14 Compare July 29, 2022 12:46
@function2-llx function2-llx changed the title [WIP] Update patch merging with a for loop Update patch merging with a for loop Jul 29, 2022
@function2-llx function2-llx marked this pull request as ready for review July 29, 2022 12:52
@wyli
Copy link
Copy Markdown
Contributor

wyli commented Jul 29, 2022

/build

@function2-llx function2-llx force-pushed the 4753-improve-patch-merging branch from 4dced1a to c49788f Compare July 29, 2022 13:09
@function2-llx
Copy link
Copy Markdown
Contributor Author

function2-llx commented Jul 29, 2022

I've come up with one issue: in case someone's using 2D Swin Transformer pre-trained model, I've swapped the order of i and j in the code. I think it's unnecessary to do the same for 3D since original implementation should be used for existed pre-trained model.

@wyli
Copy link
Copy Markdown
Contributor

wyli commented Jul 29, 2022

/build

@wyli wyli enabled auto-merge (squash) July 29, 2022 13:20
@wyli
Copy link
Copy Markdown
Contributor

wyli commented Jul 29, 2022

/build

@wyli wyli disabled auto-merge July 29, 2022 14:52
@wyli wyli enabled auto-merge (squash) July 29, 2022 14:52
@wyli wyli merged commit 288edee into Project-MONAI:dev Jul 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants