Skip to content

token_request: walk the quantification points without allocating - #962

Merged
jcelerier merged 1 commit into
masterfrom
feature/pattern-sequencer-ui
Oct 18, 2026
Merged

jcelerier merged 1 commit into
masterfrom
feature/pattern-sequencer-ui

Conversation

@jcelerier

Copy link
Copy Markdown
Member

for_each_quantification_date(rate, f) calls f with every quantification point of the tick, in the same order get_quantification_dates() lists them. get_quantification_dates() is now built on it.

A node can walk the grid on the audio thread without the small_vector<8> spilling to the heap past 8 points. Walking forward also no longer collects the bar segments into a list. Rewinding still collects them, since it walks them in reverse.

score's pattern sequencer uses this to walk its grid without allocating (ossia/score PR to follow).

🤖 Generated with Claude Code

https://claude.ai/code/session_01FCfP6rUUXFnGw6VoTiRCWB

for_each_quantification_date() calls a function with every point of the
tick, in the order get_quantification_dates() lists them, which is now
built on it. A node can walk the grid on the audio thread without the
small_vector spilling to the heap past 8 points; walking forward no longer
collects the bar segments either.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01FCfP6rUUXFnGw6VoTiRCWB
@jcelerier
jcelerier temporarily deployed to Apple Certificate October 17, 2026 21:20 — with GitHub Actions Inactive
@jcelerier
jcelerier temporarily deployed to Apple Certificate October 17, 2026 21:20 — with GitHub Actions Inactive
@jcelerier
jcelerier temporarily deployed to Apple Certificate October 17, 2026 21:20 — with GitHub Actions Inactive
@jcelerier
jcelerier temporarily deployed to Apple Certificate October 17, 2026 21:20 — with GitHub Actions Inactive
@jcelerier
jcelerier temporarily deployed to Apple Certificate October 17, 2026 21:20 — with GitHub Actions Inactive
@jcelerier
jcelerier temporarily deployed to Apple Certificate October 17, 2026 21:20 — with GitHub Actions Inactive
@jcelerier
jcelerier temporarily deployed to Apple Certificate October 17, 2026 21:20 — with GitHub Actions Inactive
@jcelerier
jcelerier temporarily deployed to Apple Certificate October 17, 2026 21:20 — with GitHub Actions Inactive
@jcelerier
jcelerier temporarily deployed to Apple Certificate October 17, 2026 21:20 — with GitHub Actions Inactive
@jcelerier
jcelerier temporarily deployed to Apple Certificate October 17, 2026 21:20 — with GitHub Actions Inactive
@jcelerier
jcelerier merged commit 70c3733 into master Oct 18, 2026
43 of 47 checks passed
@jcelerier
jcelerier deleted the feature/pattern-sequencer-ui branch October 18, 2026 16:51

This branch was previously deployed

1 inactive deployment
Apple Certificate — 3e33978c Deployed Oct 17, 2026 by jcelerier via CPP (macOS) #1273
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant