mirror of
https://github.com/zebrajr/pytorch.git
synced 2025-12-07 12:21:27 +01:00
74db92b218
Replaces 71 assert statements across 11 files in `torch.distributed` with explicit if-checks raising AssertionError to prevent assertions from being disabled with Python -O flag. Fixes #164878 Pull Request resolved: https://github.com/pytorch/pytorch/pull/165216 Approved by: https://github.com/albanD
47 lines
1.4 KiB
Python
47 lines
1.4 KiB
Python
import weakref
|
|
from typing import cast, Optional
|
|
|
|
import torch.nn as nn
|
|
|
|
|
|
class _State:
|
|
pass
|
|
|
|
|
|
_module_state_mapping: weakref.WeakKeyDictionary[
|
|
nn.Module, weakref.ReferenceType[_State]
|
|
] = weakref.WeakKeyDictionary()
|
|
|
|
|
|
def _insert_module_state(module: nn.Module, state: _State) -> None:
|
|
global _module_state_mapping
|
|
if module in _module_state_mapping:
|
|
raise AssertionError(f"Inserting {module} more than once.")
|
|
_module_state_mapping[module] = weakref.ref(state)
|
|
|
|
|
|
def _get_module_state(module: nn.Module) -> Optional[_State]:
|
|
"""
|
|
Return the ``_State`` in ``model``.
|
|
|
|
Given a ``module``, this API finds out if the module is also a ``_State``
|
|
instance or if the module is managed by a composable API. If the module
|
|
is also a ``_State``, ``module`` will be casted to ``_State` and returned.
|
|
If it is managed by a composable API, the corresponding ``_State`` will
|
|
be returned.
|
|
"""
|
|
global _module_state_mapping
|
|
if isinstance(module, _State):
|
|
# pyrefly: ignore # redundant-cast
|
|
return cast(_State, module)
|
|
else:
|
|
# https://github.com/pytorch/pytorch/issues/107054
|
|
if module in _module_state_mapping:
|
|
state_ref = _module_state_mapping[module]
|
|
state = state_ref()
|
|
if state is None:
|
|
raise AssertionError("State has already been garbage collected")
|
|
return state
|
|
else:
|
|
return None
|