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

Exit LM task if special token exists in text for ByteTensorizer #1207

Closed
wants to merge 1 commit into from

Conversation

FanW123
Copy link

@FanW123 FanW123 commented Dec 18, 2019

Summary:

  1. In Language Model task, if data contains BYTE_BOS or BYTE_EOS special tokens, exit and print out the message. This will prevent training from failing silently.
  2. Add a test

Reviewed By: kmalik22

Differential Revision: D19084083

@facebook-github-bot facebook-github-bot added CLA Signed Do not delete this pull request or issue due to inactivity. fb-exported labels Dec 18, 2019
@facebook-github-bot
Copy link
Contributor

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

FanW123 pushed a commit to FanW123/pytext that referenced this pull request Dec 18, 2019
…bookresearch#1207)

Summary:
Pull Request resolved: facebookresearch#1207

1. In Language Model task, if data contains BYTE_BOS or BYTE_EOS special tokens, exit and print out the message. This will prevent training from failing silently.
2. Add a test

Reviewed By: kmalik22

Differential Revision: D19084083

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

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

…bookresearch#1207)

Summary:
Pull Request resolved: facebookresearch#1207

1. In Language Model task, if data contains BYTE_BOS or BYTE_EOS special tokens, exit and print out the message. This will prevent training from failing silently.
2. Add a test

Reviewed By: kmalik22

Differential Revision: D19084083

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

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

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 1fb27b9.

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. fb-exported Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants