composer
composer copied to clipboard
Copy paste
Copy Paste augmentation, ready for review.
I did not delete the debug_out
folder (which contains some sample outputs of CopyPaste, in case you need them in the future. Please feel free to take it out of the PR late. @Landanjs
I think the debug folder was committed by mistake?
Going to defer to others for reviews on algorithms...
It's a useful reference to look at while reviewing, but yes we need to remember to remove all of the images before merging
I'm not sure if Daya needs to be a reviewer, but I can't remove him? Cory, Evan and I should be enough
It automatically requested a review from you @dblalock 😞
@Landanjs are we still trying to merge this? I keep getting notifications about how this needs review
I was addressing comments and trying to fix pyright in my own fork at branch landan/copy-paste
, but I couldn't finish it in time... So it might be 1.5 weeks until another update. @dblalock could you approve to prevent notifications? Austin, Cory, and Evan would be the main reviewers
Hi all, I was unable to get an improvement from CopyPaste on ADE20k (see wandb runs). I tried a few tweaks and adjusting hyperparameters, but without success. Since semantic segmentation specific speedups have lower priority now and this was designed for instance segmentation (which we don't have), we decided to merge it in the contrib repo so we can come back to the method in the future. Here is the mcontrib PR