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

open source Roberta #1032

Closed
wants to merge 2 commits into from
Closed

Conversation

borguz
Copy link
Contributor

@borguz borguz commented Oct 3, 2019

Summary: Move Roberta model to OSS

Differential Revision: D17747712

@facebook-github-bot facebook-github-bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Oct 3, 2019
@facebook-github-bot
Copy link
Contributor

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

Summary:
Pull Request resolved: facebookresearch#1017

Currently Roberta encoder, model and tensorizer are pretty stand-alone, not conforming to other PyText tasks.  This diff is an attempt to better integrate it.

It involves the following:
- Make GPT-2 BPE act like a proper tokenizer and also return char indices.  This makes Roberta tensorizer more modular so code can be re-used
- Make Roberta tensorizer conform more closely to BERTTensorizer so that the TransformerSentenceEncoder interfaces are better aligned.
- Add a Roberta tensorizer for question answering

Differential Revision: D17690805

fbshipit-source-id: e97fe1352047e8d915005a3fcdd76bd53b268a03
Summary:
Pull Request resolved: facebookresearch#1032

Move Roberta model to OSS

Reviewed By: hikushalhere

Differential Revision: D17747712

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

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

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 528c3db.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CLA Signed Do not delete this pull request or issue due to inactivity. Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants