Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

Archive detectron dependencies #4514

Merged
merged 4 commits into from
Apr 27, 2022
Merged

Archive detectron dependencies #4514

merged 4 commits into from
Apr 27, 2022

Conversation

stephenroller
Copy link
Contributor

@stephenroller stephenroller commented Apr 26, 2022 •
edited
Loading

Patch description
As per discussion in the group, deprecate the multimodal blenderbot due to its unweidly mask r-cnn dependency.

Testing steps
CI

@stephenroller stephenroller marked this pull request as ready for review April 26, 2022 19:38
klshuster
klshuster approved these changes Apr 26, 2022
View reviewed changes
Copy link
Contributor

@klshuster klshuster left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks like there are some imports you can remove from image_featurizers.py to appease lint

@stephenroller stephenroller merged commit 7a8ab4f into main Apr 27, 2022
@stephenroller stephenroller deleted the norcnn branch April 27, 2022 14:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@klshuster klshuster klshuster approved these changes

Assignees
No one assigned
Labels
CLA Signed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@stephenroller @klshuster @facebook-github-bot