Skip to content

Commit 1642459

Browse files
committed
offload support, bug fixes, remove mixins
1 parent 5ebcab3 commit 1642459

2 files changed

Lines changed: 51 additions & 25 deletions

File tree

comfy/ops.py

Lines changed: 15 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,10 @@ def cast_bias_weight(s, input=None, dtype=None, device=None, bias_dtype=None, of
7777
# will add async-offload support to your cast and improve performance.
7878
if input is not None:
7979
if dtype is None:
80-
dtype = input.dtype
80+
if isinstance(input, QuantizedTensor):
81+
dtype = input._layout_params["orig_dtype"]
82+
else:
83+
dtype = input.dtype
8184
if bias_dtype is None:
8285
bias_dtype = dtype
8386
if device is None:
@@ -534,18 +537,7 @@ def forward(self, *args, **kwargs):
534537
# ==============================================================================
535538
# Mixed Precision Operations
536539
# ==============================================================================
537-
from .quant_ops import QuantizedTensor
538-
539-
QUANT_FORMAT_MIXINS = {
540-
"float8_e4m3fn": {
541-
"dtype": torch.float8_e4m3fn,
542-
"layout_type": "TensorCoreFP8Layout",
543-
"parameters": {
544-
"weight_scale": torch.nn.Parameter(torch.zeros((), dtype=torch.float32), requires_grad=False),
545-
"input_scale": torch.nn.Parameter(torch.zeros((), dtype=torch.float32), requires_grad=False),
546-
}
547-
}
548-
}
540+
from .quant_ops import QuantizedTensor, QUANT_ALGOS
549541

550542
class MixedPrecisionOps(disable_weight_init):
551543
_layer_quant_config = {}
@@ -596,23 +588,24 @@ def _load_from_state_dict(self, state_dict, prefix, local_metadata,
596588
if quant_format is None:
597589
raise ValueError(f"Unknown quantization format for layer {layer_name}")
598590

599-
mixin = QUANT_FORMAT_MIXINS[quant_format]
600-
self.layout_type = mixin["layout_type"]
591+
qconfig = QUANT_ALGOS[quant_format]
592+
self.layout_type = qconfig["comfy_tensor_layout"]
601593

602-
scale_key = f"{prefix}weight_scale"
594+
weight_scale_key = f"{prefix}weight_scale"
603595
layout_params = {
604-
'scale': state_dict.pop(scale_key, None),
605-
'orig_dtype': MixedPrecisionOps._compute_dtype
596+
'scale': state_dict.pop(weight_scale_key, None),
597+
'orig_dtype': MixedPrecisionOps._compute_dtype,
598+
'block_size': qconfig.get("group_size", None),
606599
}
607600
if layout_params['scale'] is not None:
608-
manually_loaded_keys.append(scale_key)
601+
manually_loaded_keys.append(weight_scale_key)
609602

610603
self.weight = torch.nn.Parameter(
611-
QuantizedTensor(weight.to(device=device, dtype=mixin["dtype"]), self.layout_type, layout_params),
604+
QuantizedTensor(weight.to(device=device), self.layout_type, layout_params),
612605
requires_grad=False
613606
)
614607

615-
for param_name, param_value in mixin["parameters"].items():
608+
for param_name in qconfig["parameters"]:
616609
param_key = f"{prefix}{param_name}"
617610
_v = state_dict.pop(param_key, None)
618611
if _v is None:
@@ -643,7 +636,7 @@ def forward(self, input, *args, **kwargs):
643636
if (getattr(self, 'layout_type', None) is not None and
644637
getattr(self, 'input_scale', None) is not None and
645638
not isinstance(input, QuantizedTensor)):
646-
input = QuantizedTensor.from_float(input, self.layout_type, scale=self.input_scale, fp8_dtype=self.weight.dtype)
639+
input = QuantizedTensor.from_float(input, self.layout_type, scale=self.input_scale, dtype=self.weight.dtype)
647640
return self._forward(input, self.weight, self.bias)
648641

649642

comfy/quant_ops.py

Lines changed: 36 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -74,6 +74,12 @@ def _copy_layout_params(params):
7474
new_params[k] = v
7575
return new_params
7676

77+
def _copy_layout_params_inplace(src, dst, non_blocking=False):
78+
for k, v in src.items():
79+
if isinstance(v, torch.Tensor):
80+
dst[k].copy_(v, non_blocking=non_blocking)
81+
else:
82+
dst[k] = v
7783

7884
class QuantizedLayout:
7985
"""
@@ -318,13 +324,13 @@ def generic_to_dtype_layout(func, args, kwargs):
318324
def generic_copy_(func, args, kwargs):
319325
qt_dest = args[0]
320326
src = args[1]
321-
327+
non_blocking = args[2] if len(args) > 2 else False
322328
if isinstance(qt_dest, QuantizedTensor):
323329
if isinstance(src, QuantizedTensor):
324330
# Copy from another quantized tensor
325-
qt_dest._qdata.copy_(src._qdata)
331+
qt_dest._qdata.copy_(src._qdata, non_blocking=non_blocking)
326332
qt_dest._layout_type = src._layout_type
327-
qt_dest._layout_params = _copy_layout_params(src._layout_params)
333+
_copy_layout_params_inplace(src._layout_params, qt_dest._layout_params, non_blocking=non_blocking)
328334
else:
329335
# Copy from regular tensor - just copy raw data
330336
qt_dest._qdata.copy_(src)
@@ -336,6 +342,26 @@ def generic_copy_(func, args, kwargs):
336342
def generic_has_compatible_shallow_copy_type(func, args, kwargs):
337343
return True
338344

345+
346+
@register_generic_util(torch.ops.aten.empty_like.default)
347+
def generic_empty_like(func, args, kwargs):
348+
"""Empty_like operation - creates an empty tensor with the same quantized structure."""
349+
qt = args[0]
350+
if isinstance(qt, QuantizedTensor):
351+
# Create empty tensor with same shape and dtype as the quantized data
352+
hp_dtype = kwargs.pop('dtype', qt._layout_params["orig_dtype"])
353+
new_qdata = torch.empty_like(qt._qdata, **kwargs)
354+
355+
# Handle device transfer for layout params
356+
target_device = kwargs.get('device', new_qdata.device)
357+
new_params = _move_layout_params_to_device(qt._layout_params, target_device)
358+
359+
# Update orig_dtype if dtype is specified
360+
new_params['orig_dtype'] = hp_dtype
361+
362+
return QuantizedTensor(new_qdata, qt._layout_type, new_params)
363+
return func(*args, **kwargs)
364+
339365
# ==============================================================================
340366
# FP8 Layout + Operation Handlers
341367
# ==============================================================================
@@ -378,6 +404,13 @@ def dequantize(qdata, scale, orig_dtype, **kwargs):
378404
def get_plain_tensors(cls, qtensor):
379405
return qtensor._qdata, qtensor._layout_params['scale']
380406

407+
QUANT_ALGOS = {
408+
"float8_e4m3fn": {
409+
"storage_t": torch.float8_e4m3fn,
410+
"parameters": {"weight_scale", "input_scale"},
411+
"comfy_tensor_layout": "TensorCoreFP8Layout",
412+
},
413+
}
381414

382415
LAYOUTS = {
383416
"TensorCoreFP8Layout": TensorCoreFP8Layout,

0 commit comments

Comments
 (0)