Skip to content

Copy transformer impl to oss folder #443

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Copy transformer impl to oss folder #443

wants to merge 1 commit into from

Conversation

ankitade
Copy link
Contributor

@ankitade ankitade commented Aug 8, 2023

Differential Revision: D48132386

@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported labels Aug 8, 2023
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D48132386

@ankitade ankitade marked this pull request as draft August 9, 2023 00:02
@facebook-github-bot
Copy transformer impl to oss folder (facebookresearch#443)
5912400 
Summary: Pull Request resolved: facebookresearch#443

Differential Revision: D48132386

fbshipit-source-id: 0dcb659fe0f5698628d6e2742f256b87132d0cca
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D48132386

@codecov-commenter
Copy link

codecov-commenter commented Aug 16, 2023 •
edited
Loading

Codecov Report

Patch coverage: 98.51% and project coverage change: +0.34% 🎉

Comparison is base (a1cc8f3) 68.77% compared to head (5912400) 69.11%.
Report is 1 commits behind head on main.

Additional details and impacted files 
@@            Coverage Diff             @@
##             main     #443      +/-   ##
==========================================
+ Coverage   68.77%   69.11%   +0.34%     
==========================================
  Files         169      170       +1     
  Lines       11392    11525     +133     
==========================================
+ Hits         7835     7966     +131     
- Misses       3557     3559       +2
Files Changed Coverage Δ
torchmultimodal/modules/layers/transformer.py 97.33% <97.10%> (-2.67%) ⬇️
tests/modules/layers/test_transformer.py 100.00% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ankitade @facebook-github-bot @codecov-commenter