Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Better profile_callable #900

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Better profile_callable #900

wants to merge 1 commit into from

Commits on Aug 16, 2023

  1. Better profile_callable 

    Summary:
I extended profile_callable to accept a list of events of interest. In
the case where this parameter is not specified, profile_callable will collect
the timings of all events except those ending in " Sync". This gives us more
accurate results; we no longer measure overhead from the syncs.

Finally, I converted the softmax benchmark to use profile_callable for both the
AIT and PT2 benchmarking. Using the same profiling function ensures we are
comparing apples to apples.

Differential Revision: D48252668

fbshipit-source-id: cebc4b398e525b2db755192f3c57ab298ac42faf
    @int3 @facebook-github-bot
    int3 authored and facebook-github-bot committed Aug 16, 2023
    Configuration menu
    Copy the full SHA
    a3e1273 View commit details
    Browse the repository at this point in the history