Skip to content

Commit

Permalink
Merge pull request AUTOMATIC1111#4918 from brkirch/pytorch-fixes
Browse files Browse the repository at this point in the history
Fixes for PyTorch 1.12.1 when using MPS
  • Loading branch information
AUTOMATIC1111 authored Nov 27, 2022
2 parents 10923f9 + e247b74 commit cc90dcc
Show file tree
Hide file tree
Showing 4 changed files with 27 additions and 10 deletions.
31 changes: 24 additions & 7 deletions modules/devices.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,10 @@
import contextlib
import torch
from modules import errors
from packaging import version


# has_mps is only available in nightly pytorch (for now) and MasOS 12.3+.
# has_mps is only available in nightly pytorch (for now) and macOS 12.3+.
# check `getattr` and try it for compatibility
def has_mps() -> bool:
if not getattr(torch, 'has_mps', False):
Expand Down Expand Up @@ -99,9 +100,25 @@ def autocast(disable=False):


# MPS workaround for https://github.com/pytorch/pytorch/issues/79383
def mps_contiguous(input_tensor, device):
return input_tensor.contiguous() if device.type == 'mps' else input_tensor


def mps_contiguous_to(input_tensor, device):
return mps_contiguous(input_tensor, device).to(device)
orig_tensor_to = torch.Tensor.to
def tensor_to_fix(self, *args, **kwargs):
if self.device.type != 'mps' and \
((len(args) > 0 and isinstance(args[0], torch.device) and args[0].type == 'mps') or \
(isinstance(kwargs.get('device'), torch.device) and kwargs['device'].type == 'mps')):
self = self.contiguous()
return orig_tensor_to(self, *args, **kwargs)


# MPS workaround for https://github.com/pytorch/pytorch/issues/80800
orig_layer_norm = torch.nn.functional.layer_norm
def layer_norm_fix(*args, **kwargs):
if len(args) > 0 and isinstance(args[0], torch.Tensor) and args[0].device.type == 'mps':
args = list(args)
args[0] = args[0].contiguous()
return orig_layer_norm(*args, **kwargs)


# PyTorch 1.13 doesn't need these fixes but unfortunately is slower and has regressions that prevent training from working
if has_mps() and version.parse(torch.__version__) < version.parse("1.13"):
torch.Tensor.to = tensor_to_fix
torch.nn.functional.layer_norm = layer_norm_fix
2 changes: 1 addition & 1 deletion modules/esrgan_model.py
Original file line number Diff line number Diff line change
Expand Up @@ -199,7 +199,7 @@ def upscale_without_tiling(model, img):
img = img[:, :, ::-1]
img = np.ascontiguousarray(np.transpose(img, (2, 0, 1))) / 255
img = torch.from_numpy(img).float()
img = devices.mps_contiguous_to(img.unsqueeze(0), devices.device_esrgan)
img = img.unsqueeze(0).to(devices.device_esrgan)
with torch.no_grad():
output = model(img)
output = output.squeeze().float().cpu().clamp_(0, 1).numpy()
Expand Down
2 changes: 1 addition & 1 deletion modules/scunet_model.py
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ def do_upscale(self, img: PIL.Image, selected_file):
img = img[:, :, ::-1]
img = np.moveaxis(img, 2, 0) / 255
img = torch.from_numpy(img).float()
img = devices.mps_contiguous_to(img.unsqueeze(0), device)
img = img.unsqueeze(0).to(device)

with torch.no_grad():
output = model(img)
Expand Down
2 changes: 1 addition & 1 deletion modules/swinir_model.py
Original file line number Diff line number Diff line change
Expand Up @@ -111,7 +111,7 @@ def upscale(
img = img[:, :, ::-1]
img = np.moveaxis(img, 2, 0) / 255
img = torch.from_numpy(img).float()
img = devices.mps_contiguous_to(img.unsqueeze(0), devices.device_swinir)
img = img.unsqueeze(0).to(devices.device_swinir)
with torch.no_grad(), precision_scope("cuda"):
_, _, h_old, w_old = img.size()
h_pad = (h_old // window_size + 1) * window_size - h_old
Expand Down

0 comments on commit cc90dcc

Please sign in to comment.