Skip to content

Commit aa592c1

Browse files
[MISC] Clarify why to_numpy() still needs qd.sync() with shared queue
Co-authored-by: Cursor <cursoragent@cursor.com>
1 parent bf13284 commit aa592c1

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

docs/source/user_guide/metal_shared_queue.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -94,7 +94,7 @@ qd.init(arch=qd.metal, external_metal_command_queue=queue_ptr)
9494
| `f.to_torch(copy=False)` | `qd.sync()` called internally | no sync needed |
9595
| `f.to_torch(copy=True)` | `qd.sync()` + `torch.mps.synchronize()` | no sync needed |
9696
| Quadrants kernel after torch write | manual `torch.mps.synchronize()` required | automatic (same queue) |
97-
| `f.to_numpy()` | `qd.sync()` (always needed for CPU readback) | `qd.sync()` (still needed) |
97+
| `f.to_numpy()` | `qd.sync()` (always needed for CPU readback) | `qd.sync()` (still needed — the kernel-copy path is Quadrants-internal, not routed through MPS) |
9898

9999
## Lifetime and ownership
100100

0 commit comments

Comments
 (0)