Skip to content

Commit ed2b499

Browse files
Merge pull request #247 from menloresearch/update-dev-from-master-2025-09-10-00-32
Sync master with upstream release b6432
2 parents a153710 + ae355f6 commit ed2b499

39 files changed

+870
-424
lines changed

CONTRIBUTING.md

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,9 @@
1616
- Use the following format for the squashed commit title: `<module> : <commit title> (#<issue_number>)`. For example: `utils : fix typo in utils.py (#1234)`
1717
- Optionally pick a `<module>` from here: https://github.com/ggml-org/llama.cpp/wiki/Modules
1818
- Consider adding yourself to [CODEOWNERS](CODEOWNERS)
19+
- Let authors, who are also collaborators, merge their own PRs
20+
- When merging a PR by a contributor, make sure you have a good understanding of the changes
21+
- Be mindful of maintenance: most of the work going into a feature happens after the PR is merged. If the PR author is not committed to contribute long-term, someone else needs to take responsibility (you)
1922

2023
# Coding guidelines
2124

examples/eval-callback/eval-callback.cpp

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,15 @@ static std::string ggml_ne_string(const ggml_tensor * t) {
2828
return str;
2929
}
3030

31+
static inline float ggml_compute_bf16_to_fp32(ggml_bf16_t h) {
32+
union {
33+
float f;
34+
uint32_t i;
35+
} u;
36+
u.i = (uint32_t)h.bits << 16;
37+
return u.f;
38+
}
39+
3140
static float ggml_get_float_value(uint8_t * data, ggml_type type, const size_t * nb, size_t i0, size_t i1, size_t i2, size_t i3) {
3241
size_t i = i3 * nb[3] + i2 * nb[2] + i1 * nb[1] + i0 * nb[0];
3342
float v;
@@ -43,6 +52,8 @@ static float ggml_get_float_value(uint8_t * data, ggml_type type, const size_t *
4352
v = (float) *(int16_t *) &data[i];
4453
} else if (type == GGML_TYPE_I8) {
4554
v = (float) *(int8_t *) &data[i];
55+
} else if (type == GGML_TYPE_BF16) {
56+
v = ggml_compute_bf16_to_fp32(*(ggml_bf16_t *) &data[i]);
4657
} else {
4758
GGML_ABORT("fatal error");
4859
}
Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
--extra-index-url https://download.pytorch.org/whl/cpu
2-
torch~=2.6.0
3-
torchvision~=0.21.0
4-
transformers~=4.55.0
5-
huggingface-hub~=0.34.0
2+
torch
3+
torchvision
4+
transformers
5+
huggingface-hub
6+
accelerate

examples/model-conversion/scripts/causal/run-org-model.py

Lines changed: 141 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -9,15 +9,134 @@
99
import torch
1010
import numpy as np
1111

12-
unreleased_model_name = os.getenv('UNRELEASED_MODEL_NAME')
13-
14-
parser = argparse.ArgumentParser(description='Process model with specified path')
15-
parser.add_argument('--model-path', '-m', help='Path to the model')
12+
### If you want to dump RoPE activations, apply this monkey patch to the model
13+
### class from Transformers that you are running (replace apertus.modeling_apertus
14+
### with the proper package and class for your model
15+
### === START ROPE DEBUG ===
16+
# from transformers.models.apertus.modeling_apertus import apply_rotary_pos_emb
17+
18+
# orig_rope = apply_rotary_pos_emb
19+
# torch.set_printoptions(threshold=float('inf'))
20+
# torch.set_printoptions(precision=6, sci_mode=False)
21+
22+
# def debug_rope(q, k, cos, sin, position_ids=None, unsqueeze_dim=1):
23+
# # log inputs
24+
# summarize(q, "RoPE.q_in")
25+
# summarize(k, "RoPE.k_in")
26+
27+
# # call original
28+
# q_out, k_out = orig_rope(q, k, cos, sin, position_ids, unsqueeze_dim)
29+
30+
# # log outputs
31+
# summarize(q_out, "RoPE.q_out")
32+
# summarize(k_out, "RoPE.k_out")
33+
34+
# return q_out, k_out
35+
36+
# # Patch it
37+
# import transformers.models.apertus.modeling_apertus as apertus_mod # noqa: E402
38+
# apertus_mod.apply_rotary_pos_emb = debug_rope
39+
### == END ROPE DEBUG ===
40+
41+
42+
def summarize(tensor: torch.Tensor, name: str, max_seq: int = 3, max_vals: int = 3):
43+
"""
44+
Print a tensor in llama.cpp debug style.
45+
46+
Supports:
47+
- 2D tensors (seq, hidden)
48+
- 3D tensors (batch, seq, hidden)
49+
- 4D tensors (batch, seq, heads, dim_per_head) via flattening heads × dim_per_head
50+
51+
Shows first and last max_vals of each vector per sequence position.
52+
"""
53+
t = tensor.detach().to(torch.float32).cpu()
54+
55+
# Determine dimensions
56+
if t.ndim == 3:
57+
_, s, _ = t.shape
58+
elif t.ndim == 2:
59+
_, s = 1, t.shape[0]
60+
t = t.unsqueeze(0)
61+
elif t.ndim == 4:
62+
_, s, _, _ = t.shape
63+
else:
64+
print(f"Skipping tensor due to unsupported dimensions: {t.ndim}")
65+
return
66+
67+
ten_shape = t.shape
68+
69+
print(f"ggml_debug: {name} = (f32) ... = {{{ten_shape}}}")
70+
print(" [")
71+
print(" [")
72+
73+
# Determine indices for first and last sequences
74+
first_indices = list(range(min(s, max_seq)))
75+
last_indices = list(range(max(0, s - max_seq), s))
76+
77+
# Check if there's an overlap between first and last indices or if we're at the edge case of s = 2 * max_seq
78+
has_overlap = bool(set(first_indices) & set(last_indices)) or (max_seq * 2 == s)
79+
80+
# Combine indices
81+
if has_overlap:
82+
# If there's overlap, just use the combined unique indices
83+
indices = sorted(list(set(first_indices + last_indices)))
84+
separator_index = None
85+
else:
86+
# If no overlap, we'll add a separator between first and last sequences
87+
indices = first_indices + last_indices
88+
separator_index = len(first_indices)
89+
90+
for i, si in enumerate(indices):
91+
# Add separator if needed
92+
if separator_index is not None and i == separator_index:
93+
print(" ...")
94+
95+
# Extract appropriate slice
96+
vec = t[0, si]
97+
if vec.ndim == 2: # 4D case: flatten heads × dim_per_head
98+
flat = vec.flatten().tolist()
99+
else: # 2D or 3D case
100+
flat = vec.tolist()
101+
102+
# First and last slices
103+
first = flat[:max_vals]
104+
last = flat[-max_vals:] if len(flat) >= max_vals else flat
105+
first_str = ", ".join(f"{v:12.4f}" for v in first)
106+
last_str = ", ".join(f"{v:12.4f}" for v in last)
107+
108+
print(f" [{first_str}, ..., {last_str}]")
109+
110+
print(" ],")
111+
print(" ]")
112+
print(f" sum = {t.sum().item():.6f}\n")
113+
114+
115+
def debug_hook(name):
116+
def fn(_m, input, output):
117+
if isinstance(input, torch.Tensor):
118+
summarize(input, name + "_in")
119+
elif isinstance(input, (tuple, list)) and isinstance(input[0], torch.Tensor):
120+
summarize(input[0], name + "_in")
121+
if isinstance(output, torch.Tensor):
122+
summarize(output, name + "_out")
123+
elif isinstance(output, (tuple, list)) and isinstance(output[0], torch.Tensor):
124+
summarize(output[0], name + "_out")
125+
126+
return fn
127+
128+
129+
unreleased_model_name = os.getenv("UNRELEASED_MODEL_NAME")
130+
131+
parser = argparse.ArgumentParser(description="Process model with specified path")
132+
parser.add_argument("--model-path", "-m", help="Path to the model")
16133
args = parser.parse_args()
17134

18-
model_path = os.environ.get('MODEL_PATH', args.model_path)
135+
model_path = os.environ.get("MODEL_PATH", args.model_path)
19136
if model_path is None:
20-
parser.error("Model path must be specified either via --model-path argument or MODEL_PATH environment variable")
137+
parser.error(
138+
"Model path must be specified either via --model-path argument or MODEL_PATH environment variable"
139+
)
21140

22141
config = AutoConfig.from_pretrained(model_path)
23142

@@ -34,18 +153,30 @@
34153

35154
if unreleased_model_name:
36155
model_name_lower = unreleased_model_name.lower()
37-
unreleased_module_path = f"transformers.models.{model_name_lower}.modular_{model_name_lower}"
156+
unreleased_module_path = (
157+
f"transformers.models.{model_name_lower}.modular_{model_name_lower}"
158+
)
38159
class_name = f"{unreleased_model_name}ForCausalLM"
39160
print(f"Importing unreleased model module: {unreleased_module_path}")
40161

41162
try:
42-
model_class = getattr(importlib.import_module(unreleased_module_path), class_name)
43-
model = model_class.from_pretrained(model_path) # Note: from_pretrained, not fromPretrained
163+
model_class = getattr(
164+
importlib.import_module(unreleased_module_path), class_name
165+
)
166+
model = model_class.from_pretrained(
167+
model_path
168+
) # Note: from_pretrained, not fromPretrained
44169
except (ImportError, AttributeError) as e:
45170
print(f"Failed to import or load model: {e}")
46171
exit(1)
47172
else:
48-
model = AutoModelForCausalLM.from_pretrained(model_path)
173+
model = AutoModelForCausalLM.from_pretrained(
174+
model_path, device_map="auto", offload_folder="offload"
175+
)
176+
177+
for name, module in model.named_modules():
178+
if len(list(module.children())) == 0: # only leaf modules
179+
module.register_forward_hook(debug_hook(name))
49180

50181
model_name = os.path.basename(model_path)
51182
# Printing the Model class to allow for easier debugging. This can be useful

ggml/src/ggml-cuda/CMakeLists.txt

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,8 @@ if (CUDAToolkit_FOUND)
4444
list(APPEND GGML_SOURCES_CUDA ${SRCS})
4545
file(GLOB SRCS "template-instances/mmq*.cu")
4646
list(APPEND GGML_SOURCES_CUDA ${SRCS})
47+
file(GLOB SRCS "template-instances/mmf*.cu")
48+
list(APPEND GGML_SOURCES_CUDA ${SRCS})
4749

4850
if (GGML_CUDA_FA_ALL_QUANTS)
4951
file(GLOB SRCS "template-instances/fattn-vec*.cu")

ggml/src/ggml-cuda/common.cuh

Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -545,6 +545,31 @@ static __device__ __forceinline__ int ggml_cuda_dp4a(const int a, const int b, i
545545
#endif // defined(GGML_USE_HIP)
546546
}
547547

548+
static __device__ __forceinline__ void ggml_cuda_mad(float & acc, const float v, const float u) {
549+
acc += v*u;
550+
}
551+
552+
static __device__ __forceinline__ void ggml_cuda_mad(float & acc, const float2 v, const float2 u) {
553+
acc += v.x*u.x;
554+
acc += v.y*u.y;
555+
}
556+
557+
static __device__ __forceinline__ void ggml_cuda_mad(float & acc, const half2 v, const half2 u) {
558+
#if defined(GGML_USE_HIP) && defined(GCN)
559+
asm volatile("v_dot2_f32_f16 %0, %1, %2, %0" : "+v"(acc) : "v"(v), "v"(u));
560+
#else
561+
#ifdef FAST_FP16_AVAILABLE
562+
const float2 tmp = __half22float2(v*u);
563+
acc += tmp.x + tmp.y;
564+
#else
565+
const float2 tmpv = __half22float2(v);
566+
const float2 tmpu = __half22float2(u);
567+
acc += tmpv.x * tmpu.x;
568+
acc += tmpv.y * tmpu.y;
569+
#endif // FAST_FP16_AVAILABLE
570+
#endif // defined(GGML_USE_HIP) && defined(GCN)
571+
}
572+
548573
static __device__ __forceinline__ float ggml_cuda_e8m0_to_fp32(uint8_t x) {
549574
#if CUDART_VERSION >= 12080
550575
const nv_bfloat16 e = __nv_cvt_e8m0_to_bf16raw(x);

ggml/src/ggml-cuda/fattn-tile.cu

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -304,12 +304,7 @@ static __global__ void flash_attn_tile(
304304
for (int i_KQ_0 = 0; i_KQ_0 < kq_stride; i_KQ_0 += warp_size) {
305305
#pragma unroll
306306
for (int j_KQ_0 = 0; j_KQ_0 < ncols; j_KQ_0 += nwarps) {
307-
#ifdef FAST_FP16_AVAILABLE
308-
const float2 tmp = __half22float2(K_k[i_KQ_0/warp_size] * Q_k[j_KQ_0/nwarps]);
309-
sum[i_KQ_0/warp_size][j_KQ_0/nwarps] += tmp.x + tmp.y;
310-
#else
311-
sum[i_KQ_0/warp_size][j_KQ_0/nwarps] += K_k[i_KQ_0/warp_size] * Q_k[j_KQ_0/nwarps];
312-
#endif // FAST_FP16_AVAILABLE
307+
ggml_cuda_mad(sum[i_KQ_0/warp_size][j_KQ_0/nwarps], K_k[i_KQ_0/warp_size], Q_k[j_KQ_0/nwarps]);
313308
}
314309
}
315310
}

0 commit comments

Comments
 (0)