Skip to content

Commit

Permalink
Add IProfiler protocol (pytorch#599)
Browse files Browse the repository at this point in the history
Summary:

Add a protocol to define a Profiler.

Reviewed By: crassirostris

Differential Revision: D50765964
  • Loading branch information
Danielle Pintz authored and facebook-github-bot committed Oct 30, 2023
1 parent d3fdc28 commit ed67352
Show file tree
Hide file tree
Showing 2 changed files with 42 additions and 0 deletions.
2 changes: 2 additions & 0 deletions torchtnt/utils/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,7 @@
prepare_ddp,
prepare_fsdp,
)
from .profiler import IProfiler
from .progress import Progress
from .rank_zero_log import (
rank_zero_critical,
Expand Down Expand Up @@ -121,6 +122,7 @@
"NOOPStrategy",
"prepare_ddp",
"prepare_fsdp",
"IProfiler",
"Progress",
"rank_zero_critical",
"rank_zero_debug",
Expand Down
40 changes: 40 additions & 0 deletions torchtnt/utils/profiler.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,40 @@
# Copyright (c) Meta Platforms, Inc. and affiliates.
# All rights reserved.
#
# This source code is licensed under the BSD-style license found in the
# LICENSE file in the root directory of this source tree.

import logging
from types import TracebackType
from typing import Optional, Protocol, Type

logger: logging.Logger = logging.getLogger(__name__)


class IProfiler(Protocol):
"""Protocol for profilers. Can be used as a context manager."""

def __enter__(self) -> None:
"""Enters the context manager and starts the profiler."""
pass

def __exit__(
self,
exc_type: Optional[Type[BaseException]],
exc_value: Optional[BaseException],
tb: Optional[TracebackType],
) -> Optional[bool]:
"""Exits the context manager and stops the profiler."""
pass

def start(self) -> None:
"""Starts the profiler."""
pass

def stop(self) -> None:
"""Stops the profiler."""
pass

def step(self) -> None:
"""Signals to the profiler that the next profiling step has started."""
pass

0 comments on commit ed67352

Please sign in to comment.