Skip to content

Commit 084a7dd

Browse files
committed
Refactor imports in precision and strategy modules for consistency
1 parent b240864 commit 084a7dd

File tree

4 files changed

+7
-8
lines changed

4 files changed

+7
-8
lines changed

src/lightning/fabric/plugins/precision/deepspeed.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,13 +12,13 @@
1212
# See the License for the specific language governing permissions and
1313
# limitations under the License.
1414
from contextlib import AbstractContextManager, nullcontext
15-
from typing import TYPE_CHECKING, Any, Literal
15+
from typing import TYPE_CHECKING, Any, Literal, get_args
1616

1717
import torch
1818
from lightning_utilities.core.apply_func import apply_to_collection
1919
from torch import Tensor
2020
from torch.nn import Module
21-
from typing_extensions import get_args, override
21+
from typing_extensions import override
2222

2323
from lightning.fabric.plugins.precision.precision import Precision
2424
from lightning.fabric.plugins.precision.utils import _convert_fp_tensor, _DtypeContextManager

src/lightning/fabric/plugins/precision/xla.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,10 +12,10 @@
1212
# See the License for the specific language governing permissions and
1313
# limitations under the License.
1414
import os
15-
from typing import Any, Literal
15+
from typing import Any, Literal, get_args
1616

1717
import torch
18-
from typing_extensions import get_args, override
18+
from typing_extensions import override
1919

2020
from lightning.fabric.accelerators.xla import _XLA_AVAILABLE
2121
from lightning.fabric.plugins.precision.precision import Precision

src/lightning/fabric/strategies/xla_fsdp.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@
1616
from contextlib import AbstractContextManager, ExitStack, nullcontext
1717
from functools import partial
1818
from pathlib import Path
19-
from typing import TYPE_CHECKING, Any, Literal, Optional, Union
19+
from typing import TYPE_CHECKING, Any, Literal, Optional
2020

2121
import torch
2222
from torch import Tensor

tests/tests_pytorch/utilities/test_model_helpers.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -42,11 +42,10 @@ def test_is_overridden():
4242
reason="This test is ONLY relevant for the UNIFIED package",
4343
)
4444
def test_mixed_imports_unified():
45-
from pytorch_lightning.callbacks import EarlyStopping as OldEarlyStopping
46-
from pytorch_lightning.demos.boring_classes import BoringModel as OldBoringModel
47-
4845
from lightning.pytorch.utilities.compile import _maybe_unwrap_optimized as new_unwrap
4946
from lightning.pytorch.utilities.model_helpers import is_overridden as new_is_overridden
47+
from pytorch_lightning.callbacks import EarlyStopping as OldEarlyStopping
48+
from pytorch_lightning.demos.boring_classes import BoringModel as OldBoringModel
5049

5150
model = OldBoringModel()
5251
with pytest.raises(TypeError, match=r"`pytorch_lightning` object \(BoringModel\) to a `lightning.pytorch`"):

0 commit comments

Comments
 (0)