-
Notifications
You must be signed in to change notification settings - Fork 6
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
VIT Decoder updates #339
VIT Decoder updates #339
Conversation
super().__init__() | ||
self.num_heads = num_heads | ||
head_dim = decoder_dim // num_heads | ||
# NOTE scale factor was wrong in my original version, can set manually to be compat with prev weights |
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.
is this your note?
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.
no, I can remove that
import torch | ||
from einops.layers.torch import Rearrange | ||
from monai.networks.blocks import UnetOutBlock, UnetResBlock, UpSample | ||
|
||
from cyto_dl.nn.vits.mae import MAE_Encoder | ||
|
||
|
||
class SupperresDecoder(torch.nn.Module): | ||
class SuperresDecoder(torch.nn.Module): |
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.
this is unrelated to crossMAE but just in this PR?
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.
oops, started as a general decoder update, I can rename the pr
What does this PR do?
Before submitting
pytest
command?pre-commit run -a
command?Did you have fun?
Make sure you had fun coding 🙃