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

Provide row data even for packedLM #1031

Closed
wants to merge 1 commit into from

Conversation

Atry
Copy link
Contributor

@Atry Atry commented Oct 3, 2019

Summary: The reporter need the original text to generate the utterance column. See

self.UTTERANCE_COLUMN: [
row.get(self.RAW_TEXT_COLUMN) for row in raw_batch
]

Differential Revision: D17738252

@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: D17738252

@Atry Atry force-pushed the export-D17738252 branch from 69f850b to e2247df Compare October 3, 2019 23:47
Atry pushed a commit to Atry/pytext that referenced this pull request Oct 3, 2019
Summary:
Pull Request resolved: facebookresearch#1031

The reporter need the original text to generate the utterance column. See https://our.intern.facebook.com/intern/diffusion/FBS/browse/master/fbcode/pytext/metric_reporters/language_model_metric_reporter.py?lines=127

Reviewed By: borguz

Differential Revision: D17738252

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

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

Atry pushed a commit to Atry/pytext that referenced this pull request Oct 3, 2019
Summary:
Pull Request resolved: facebookresearch#1031

The reporter need the original text to generate the utterance column. See https://our.intern.facebook.com/intern/diffusion/FBS/browse/master/fbcode/pytext/metric_reporters/language_model_metric_reporter.py?lines=127

Reviewed By: borguz

Differential Revision: D17738252

fbshipit-source-id: 65ccd387473fd27cb9dd9d5f9e3c6d4f85463c30
@Atry Atry force-pushed the export-D17738252 branch from e2247df to b6a143a Compare October 3, 2019 23:49
@facebook-github-bot
Copy link
Contributor

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

Atry pushed a commit to Atry/pytext that referenced this pull request Oct 3, 2019
Summary:
Pull Request resolved: facebookresearch#1031

The reporter need the original text to generate the utterance column. See https://our.intern.facebook.com/intern/diffusion/FBS/browse/master/fbcode/pytext/metric_reporters/language_model_metric_reporter.py?lines=127

Reviewed By: borguz

Differential Revision: D17738252

fbshipit-source-id: 46f548452085eded5f6a744dd2f1894d59b3ae79
@Atry Atry force-pushed the export-D17738252 branch from b6a143a to 38cb9ba Compare October 3, 2019 23:56
@facebook-github-bot
Copy link
Contributor

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

Atry pushed a commit to Atry/pytext that referenced this pull request Oct 4, 2019
Summary:
Pull Request resolved: facebookresearch#1031

The reporter need the original text to generate the utterance column. See https://our.intern.facebook.com/intern/diffusion/FBS/browse/master/fbcode/pytext/metric_reporters/language_model_metric_reporter.py?lines=127

Reviewed By: borguz

Differential Revision: D17738252

fbshipit-source-id: 143baa41ef80b174eab3007b8d4e21e2e8142cac
@Atry Atry force-pushed the export-D17738252 branch from 38cb9ba to 67a82a0 Compare October 4, 2019 00:00
@facebook-github-bot
Copy link
Contributor

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

Summary:
Pull Request resolved: facebookresearch#1031

The reporter need the original text to generate the utterance column. See https://our.intern.facebook.com/intern/diffusion/FBS/browse/master/fbcode/pytext/metric_reporters/language_model_metric_reporter.py?lines=127

Reviewed By: borguz

Differential Revision: D17738252

fbshipit-source-id: 421f8b0df7ca48e86ef9f92d7c25483d89b48ff7
@Atry Atry force-pushed the export-D17738252 branch from 67a82a0 to 83422cb Compare October 4, 2019 00:02
@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 4d09e80.

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