Skip to content

Commit

Permalink
minor syntax fixes and fmt
Browse files Browse the repository at this point in the history
  • Loading branch information
achew010 committed Sep 11, 2024
1 parent 0303497 commit 3cccc41
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 6 deletions.
2 changes: 1 addition & 1 deletion tests/acceleration/test_acceleration_framework.py
Original file line number Diff line number Diff line change
Expand Up @@ -434,7 +434,7 @@ def test_framework_intialized_properly_foak():
fusedops_kernels_config = FusedOpsAndKernelsConfig(
fused_lora=FusedLoraConfig(base_layer="auto_gptq", fused_lora=True),
fast_kernels=FastKernelsConfig(
fast_loss=True, fast_rsm_layernorm=True, fast_rope_embeddings=True
fast_loss=True, fast_rms_layernorm=True, fast_rope_embeddings=True
),
)

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -129,12 +129,10 @@ def _verify_configured_dataclasses(self):

# Check that fused lora must be activated with either auto_gptq or bitsandbytes
if self.fused_lora is not None:
assert (
self.bitsandbytes is not None or self.auto_gptq is not None
), "`--fused_lora` must be accompanied by a quantized base layer"\
assert self.bitsandbytes is not None or self.auto_gptq is not None, (
"`--fused_lora` must be accompanied by a quantized base layer"
" `--auto_gptq` or `--bitsandbytes`."


)

@staticmethod
def from_dataclasses(*dataclasses: Type):
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -70,6 +70,7 @@ class FusedOpsAndKernelsConfig:
# fast kernels
fast_kernels: FastKernelsConfig = None

def __post_init__(self):

# ensure nested dataclasses initialized
ensure_nested_dataclasses_initialized(self)

0 comments on commit 3cccc41

Please sign in to comment.