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

Marginally faster display data #3544

Merged
merged 3 commits into from
Mar 31, 2021
Merged

Marginally faster display data #3544

merged 3 commits into from
Mar 31, 2021

Conversation

stephenroller
Copy link
Contributor

Patch description
Marginally faster response time for parlai dd by avoiding evaluation.

Testing steps
time parlai dd -t my_internal_teacher

Copy link
Contributor

@spencerp spencerp left a comment

Choose a reason for hiding this comment

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

Every millisecond counts!

@@ -66,7 +66,8 @@ def display_data(opt):

# create repeat label agent and assign it to the specified task
Copy link
Contributor

Choose a reason for hiding this comment

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

nit; update comment

@stephenroller stephenroller merged commit ea1b494 into master Mar 31, 2021
@stephenroller stephenroller deleted the fasterdd branch March 31, 2021 19:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants