Skip to content

Test perf#191

Merged
sneakers-the-rat merged 10 commits into
mainfrom
perf-tests
Apr 30, 2026
Merged

Test perf#191
sneakers-the-rat merged 10 commits into
mainfrom
perf-tests

Conversation

@sneakers-the-rat
Copy link
Copy Markdown
Collaborator

@sneakers-the-rat sneakers-the-rat commented Apr 29, 2026

Fix: #172

  • raise default stdout log level to ERROR in tests, which is hella expensive and unused
  • add a n_frame param to capture to only capture a few frames when that's all we care about
  • decrease the timeout before we kill capture processes

📚 Documentation preview 📚: https://miniscope-io--191.org.readthedocs.build/en/191/

@sneakers-the-rat
Copy link
Copy Markdown
Collaborator Author

alright increased test perf mostly by disabling logs and not reading things unnecessarily.

like ~2m improvement.

the biggest perf barrier remaining is just the time it takes to a) spawn processes and b) that takes a long time because our import time takes eons because of pydantic.

so once we transition to noob we can have a switch for running the streamdaq synchronously for testing (and i'm also not sure how much of a perf benefit multiprocessing is here anyway) then we can bring that down further.

@sneakers-the-rat sneakers-the-rat merged commit ef474eb into main Apr 30, 2026
32 checks passed
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.

unslow the tests

1 participant