-
Notifications
You must be signed in to change notification settings - Fork 328
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
Add the Segment Anything Model to KerasCV #1987
Changes from 5 commits
ddebecd
025dd15
43b0f2b
ac7f30e
360baf7
6cb8c2f
d7a1eb3
ada6040
93eb78f
d908527
07cb9d6
343557a
19312e7
e5767d8
8ad94ae
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -17,6 +17,10 @@ | |
from tensorflow.keras.layers import RandomWidth | ||
|
||
from keras_cv.layers.augmenter import Augmenter | ||
from keras_cv.layers.detectron2_layers import AddPositionalEmbedding | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Since this would be exported as part of the public API - we have There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Wouldn't adding a new layer in a pre-existing class invalidate the weights set for the ViT model? Also, since Although, I'd add a comment about this as a TODO so we don't forget to do it in the future. What do you think? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Done. |
||
from keras_cv.layers.detectron2_layers import MultiHeadAttentionWithRelativePE | ||
from keras_cv.layers.detectron2_layers import ViTDetPatchingAndEmbedding | ||
from keras_cv.layers.detectron2_layers import WindowedTransformerEncoder | ||
from keras_cv.layers.feature_pyramid import FeaturePyramid | ||
from keras_cv.layers.fusedmbconv import FusedMBConvBlock | ||
from keras_cv.layers.mbconv import MBConvBlock | ||
|
@@ -124,6 +128,7 @@ | |
from keras_cv.layers.regularization.dropblock_2d import DropBlock2D | ||
from keras_cv.layers.regularization.squeeze_excite import SqueezeAndExcite2D | ||
from keras_cv.layers.regularization.stochastic_depth import StochasticDepth | ||
from keras_cv.layers.serializable_sequential import SerializableSequential | ||
from keras_cv.layers.spatial_pyramid import SpatialPyramidPooling | ||
from keras_cv.layers.transformer_encoder import TransformerEncoder | ||
from keras_cv.layers.vit_layers import PatchingAndEmbedding |
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.
Do we want to put these under a
detectron2
namespace?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.
Done.