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

Add gen_n_toks to metrics documentation #4509

Merged
merged 1 commit into from
Apr 26, 2022
Merged

Add gen_n_toks to metrics documentation #4509

merged 1 commit into from
Apr 26, 2022

Conversation

Rebecca-Qian
Copy link
Contributor

Patch description
As in title. Adds documentation for gen_n_toks to https://parl.ai/docs/tutorial_metrics.html#list-of-metrics

EricMichaelSmith
EricMichaelSmith approved these changes Apr 25, 2022
View reviewed changes
Copy link
Contributor

@EricMichaelSmith EricMichaelSmith left a comment

Choose a reason for hiding this comment

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

Hmm I don't have context if this is the right definition of the metric, but happy to stamp

stephenroller
stephenroller approved these changes Apr 25, 2022
View reviewed changes
Copy link
Contributor

@stephenroller stephenroller left a comment

Choose a reason for hiding this comment

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

Ty!

@Rebecca-Qian Rebecca-Qian merged commit 15db609 into main Apr 26, 2022
@Rebecca-Qian Rebecca-Qian deleted the rebeccaqian/add_gen_n_toks_documentation branch April 26, 2022 00:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@stephenroller stephenroller stephenroller approved these changes

@EricMichaelSmith EricMichaelSmith EricMichaelSmith 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.
4 participants
@Rebecca-Qian @stephenroller @EricMichaelSmith @facebook-github-bot