-
Notifications
You must be signed in to change notification settings - Fork 607
Add support for delegate profiling in event_tracer #408
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.
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
Conversation
This pull request was exported from Phabricator. Differential Revision: D49360145 |
This pull request was exported from Phabricator. Differential Revision: D49360145 |
cbce03c
to
e28b1d6
Compare
This pull request was exported from Phabricator. Differential Revision: D49360145 |
e28b1d6
to
6430185
Compare
This pull request was exported from Phabricator. Differential Revision: D49360145 |
6430185
to
16a05c5
Compare
This pull request was exported from Phabricator. Differential Revision: D49360145 |
16a05c5
to
c6251f1
Compare
This pull request was exported from Phabricator. Differential Revision: D49360145 |
c6251f1
to
879a379
Compare
This pull request was exported from Phabricator. Differential Revision: D49360145 |
879a379
to
b7287ef
Compare
This pull request was exported from Phabricator. Differential Revision: D49360145 |
b7287ef
to
e473cbc
Compare
✅ Deploy Preview for resplendent-gnome-14e531 canceled.
|
This pull request was exported from Phabricator. Differential Revision: D49360145 |
e473cbc
to
0332652
Compare
This pull request was exported from Phabricator. Differential Revision: D49360145 |
0332652
to
0128ead
Compare
This pull request was exported from Phabricator. Differential Revision: D49360145 |
0cd9bfd
to
6739191
Compare
This pull request was exported from Phabricator. Differential Revision: D49360145 |
Summary: This diff should most probably be the final event_tracer diff that adds the user-facing API's for delegate profiling. `event_tracer_hooks_delegate.h` can be included by delegate authors in their backend runtime code to get access to these interfaces that will enable them to do profiling logging currently and debugging logging in the future. - Made changes to `event_tracer.h` to add the hooks to the `EventTracer` class. - Added `event_tracer_hooks_delegate.h` that contains the public interfaces that delegate authors can call into. Reviewed By: dbort Differential Revision: D49360145
6739191
to
e122e0e
Compare
This pull request was exported from Phabricator. Differential Revision: D49360145 |
This pull request has been merged in 7a1b7f4. |
* Refactor tokenizer Summary: Prepare to add tiktoken Test Plan: Rely on CI jobs Reviewers: Subscribers: Tasks: Tags: * Fix aoti Summary: Test Plan: Reviewers: Subscribers: Tasks: Tags: * Fix more aoti Summary: Test Plan: Reviewers: Subscribers: Tasks: Tags: * Fix tokenizer read format Summary: Test Plan: Reviewers: Subscribers: Tasks: Tags:
Summary:
This diff should most probably be the final event_tracer diff that adds the user-facing API's for delegate profiling.
event_tracer_hooks_delegate.h
can be included by delegate authors in their backend runtime code to get access to these interfaces that will enable them to do profiling logging currently and debugging logging in the future.event_tracer.h
to add the hooks to theEventTracer
class.event_tracer_hooks_delegate.h
that contains the public interfaces that delegate authors can call into.Differential Revision: D49360145