Skip to content

fix: use combine(..., axis="t") in optogenetic demo notebook#11

Merged
hinderling merged 1 commit into
pertzlab:mainfrom
hinderling:fix/optogenetic-demo-combine
May 15, 2026
Merged

fix: use combine(..., axis="t") in optogenetic demo notebook#11
hinderling merged 1 commit into
pertzlab:mainfrom
hinderling:fix/optogenetic-demo-combine

Conversation

@hinderling

Copy link
Copy Markdown
Collaborator

Summary

  • Replace removed RTMSequence + operator with combine(..., axis="t") in the two multi-phase cells of experiments/02_demo_sim_optogenetic/demo_sim_optogenetic.ipynb.
  • Update surrounding markdown to document the new API.
  • Add combine to the import line.

The other 11 notebooks that mention RTMSequence already use combine() — this was the last holdout.

Test plan

  • Notebook runs end-to-end with the virtual-microscope extra installed (uv sync --extra virtual-microscope --python 3.12); outputs regenerated.

Replace removed RTMSequence + operator with combine() in the two cells
that build multi-phase experiments, and update the surrounding markdown.
@hinderling hinderling merged commit afab7c1 into pertzlab:main May 15, 2026
@hinderling hinderling deleted the fix/optogenetic-demo-combine branch May 15, 2026 15:10
hinderling added a commit to hinderling/faro that referenced this pull request Jun 1, 2026
…ombine

fix: use combine(..., axis="t") in optogenetic demo notebook
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