Fix UpSample issue and enhance UpCat#2766
Conversation
Signed-off-by: Yiheng Wang <[email protected]>
|
Hi @ericspod @wyli @Nic-Ma , could you please help to look at this draft PR? The forward function of However, this implementation does not meet the requirements of torchscript. Do you have any suggestions to fix this issue? Thanks! |
|
Hi @yiheng-wang-nv the basic unet is only a simple implementation with a focus on the code readability for the basic architecture, I feel this PR will add a bit too much complexity I think you can create a separate standalone module called |
Signed-off-by: Yiheng Wang <[email protected]>
Signed-off-by: Yiheng Wang <[email protected]>
Thanks @wyli for the suggestion, I removed the decoder and I will submit a new PR for the implementation. Now this PR is only used to fix #2759 #2760 and add some upsample related parameters into |
UpSample issue and enhance UpCat
Signed-off-by: Yiheng Wang <[email protected]>
UpSample issue and enhance UpCatUpSample issue and enhance UpCat
Signed-off-by: Yiheng Wang [email protected]
Description
This PR fixes a fews issues mentioned in
Fixes #2759 , fixes #2760 and add some useful parameters for
UpCatclass.Status
Ready
Types of changes
./runtests.sh -f -u --net --coverage../runtests.sh --quick --unittests.make htmlcommand in thedocs/folder.