Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ Misc ] fbgemm checkpoints #6559

Merged
merged 39 commits into from
Jul 20, 2024
Merged
Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
39 commits
Select commit Hold shift + click to select a range
2426e29
stash
robertgshaw2-redhat Jul 18, 2024
7665b7b
format
robertgshaw2-redhat Jul 18, 2024
ef27613
tweak arg name
robertgshaw2-redhat Jul 18, 2024
2f96157
fix test
robertgshaw2-redhat Jul 18, 2024
e748554
format
robertgshaw2-redhat Jul 18, 2024
3ef571b
working e2e with our cutlass kernels
robertgshaw2-redhat Jul 19, 2024
ad83666
added fp8 gemm
robertgshaw2-redhat Jul 19, 2024
eb7d48c
remove
robertgshaw2-redhat Jul 19, 2024
90bd839
format
robertgshaw2-redhat Jul 19, 2024
15cc823
Merge branch 'main' into turn-on-fp8-dyn-per-token
robertgshaw2-redhat Jul 19, 2024
d064dd7
stash
robertgshaw2-redhat Jul 19, 2024
6aa37e5
dynamic per token
robertgshaw2-redhat Jul 19, 2024
c9d819a
format
robertgshaw2-redhat Jul 19, 2024
08cbaf7
reenable cutlass
robertgshaw2-redhat Jul 19, 2024
f4cdda1
cleanup comment
robertgshaw2-redhat Jul 19, 2024
2971f4d
format
robertgshaw2-redhat Jul 19, 2024
b601033
added dynamic per token test case
robertgshaw2-redhat Jul 19, 2024
5d8edf9
Merge branch 'turn-on-fp8-dyn-per-token' into fbgemm-checkpoints
robertgshaw2-redhat Jul 19, 2024
8b5d638
added use per token
robertgshaw2-redhat Jul 19, 2024
006ccf0
format
Jul 19, 2024
1884acf
format
Jul 19, 2024
fe14072
Make optional ubs none
Jul 19, 2024
254dcff
format
Jul 19, 2024
919d866
Merge branch 'fp8-dpt-fpgemm' into fbgemm-checkpoints
robertgshaw2-redhat Jul 19, 2024
227a277
hook up end to end with varun's ub quant kernel
robertgshaw2-redhat Jul 19, 2024
951834a
formatted
robertgshaw2-redhat Jul 19, 2024
9aa66d3
updated for nonuniform
robertgshaw2-redhat Jul 19, 2024
458a410
formatting after passing prefix around
robertgshaw2-redhat Jul 19, 2024
278f6d6
Merge branch 'main' into fbgemm-checkpoints
robertgshaw2-redhat Jul 19, 2024
3e4aaad
fixed bad merge
robertgshaw2-redhat Jul 19, 2024
de2a764
updated message
robertgshaw2-redhat Jul 20, 2024
268fe94
Merge branch 'main' into fbgemm-checkpoints
robertgshaw2-redhat Jul 20, 2024
c88fe34
merged varun's pr
robertgshaw2-redhat Jul 20, 2024
bb02a3f
fixed
robertgshaw2-redhat Jul 20, 2024
1c8f71c
cleanup pr
robertgshaw2-redhat Jul 20, 2024
6970e50
Update config.py
robertgshaw2-redhat Jul 20, 2024
94617f0
fixed config
robertgshaw2-redhat Jul 20, 2024
f9d569c
updated for new ckpt format, turned on ada lovelace, and added test case
robertgshaw2-redhat Jul 20, 2024
ae45615
format
robertgshaw2-redhat Jul 20, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,8 @@ tasks:
- name: "gsm8k"
mgoin marked this conversation as resolved.
Show resolved Hide resolved
metrics:
- name: "exact_match,strict-match"
value: 0.769
value: 0.752
- name: "exact_match,flexible-extract"
value: 0.769
value: 0.754
limit: 1000
num_fewshot: 5
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
# bash .buildkite/lm-eval-harness/run-lm-eval-gsm-vllm-baseline.sh -m nm-testing/Meta-Llama-3-8B-Instruct-FBGEMM-nonuniform -b auto -l 1000 -f 5 -t 1
model_name: "nm-testing/Meta-Llama-3-8B-Instruct-FBGEMM-nonuniform"
tasks:
- name: "gsm8k"
metrics:
- name: "exact_match,strict-match"
value: 0.753
- name: "exact_match,flexible-extract"
value: 0.753
limit: 1000
num_fewshot: 5
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,6 @@ while getopts "m:b:l:f:t:" OPT; do
done

lm_eval --model vllm \
--model_args pretrained=$MODEL,tensor_parallel_size=$TP_SIZE,add_bos_token=true,distributed_executor_backend="ray",trust_remote_code=true,max_model_len=4096 \
--model_args pretrained=$MODEL,tensor_parallel_size=$TP_SIZE,distributed_executor_backend="ray",trust_remote_code=true,max_model_len=4096 \
--tasks gsm8k --num_fewshot $FEWSHOT --limit $LIMIT \
--batch_size $BATCH_SIZE
20 changes: 9 additions & 11 deletions vllm/model_executor/layers/quantization/fbgemm_fp8.py
Original file line number Diff line number Diff line change
Expand Up @@ -27,8 +27,9 @@
class FBGEMMFp8Config(QuantizationConfig):
"""Config class for FBGEMM Fp8."""

def __init__(self, ignore_list: List[str]):
def __init__(self, ignore_list: List[str], input_scale_ub: float):
self.ignore_list = ignore_list
self.input_scale_ub = input_scale_ub

@classmethod
def get_name(cls) -> str:
Expand All @@ -40,7 +41,7 @@ def get_supported_act_dtypes(cls) -> List[torch.dtype]:

@classmethod
def get_min_capability(cls) -> int:
return 90
return 89

@classmethod
def get_config_filenames(cls) -> List[str]:
Expand All @@ -49,7 +50,8 @@ def get_config_filenames(cls) -> List[str]:
@classmethod
def from_config(cls, config: Dict[str, Any]) -> "FBGEMMFp8Config":
ignore_list = cls.get_from_keys(config, ["modules_to_not_convert"])
return cls(ignore_list=ignore_list)
input_scale_ub = cls.get_from_keys(config, ["activation_scale_ub"])
return cls(ignore_list=ignore_list, input_scale_ub=input_scale_ub)

def _is_layer_skipped(self, prefix: str) -> bool:
# prefix: model.layers.0.self_attn.q_proj
Expand Down Expand Up @@ -132,14 +134,10 @@ def create_weights(
layer.register_parameter("weight_scale", weight_scale)

# INPUT SCALE UPPER BOUND
input_scale_ub = torch.nn.Parameter(torch.zeros((1),
dtype=torch.float32),
requires_grad=False)
layer.register_parameter("input_scale_ub", input_scale_ub)
set_weight_attrs(input_scale_ub, {
"ignore_warning": True,
**extra_weight_attrs
})
input_scale_ub = torch.nn.Parameter(
torch.tensor((self.quant_config.input_scale_ub), dtype=torch.float32),
requires_grad=False)
layer.input_scale_ub = input_scale_ub

def process_weights_after_loading(self, layer: Module) -> None:
weight = layer.weight
Expand Down
Loading