SpikeInterface / SpikeInterface/spikeinterface

Compute principal components slow on Windows

Open
#3,398 6 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

continuous integration performance qualitymetrics
Dominant language
Python
Stars
847
Forks
280
Avg merge
3d 9h
Merged PRs (30d)
29

Description

#3249

Based on @chrishalcrow testing computing PCA on windows is an extremely slow step in our testing. I know the current implementation goes straight to ProcessPoolExecutor so maybe we need to revisit this and I can test locally on Windows? @alejoe91 ?

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by locating the PCA implementation and its use of ProcessPoolExecutor, then reproduce the slow computation on Windows and compare it with another platform. The issue does not name a file or test; done should mean that PCA completes without the reported Windows slowdown and that the relevant tests or benchmarks pass.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
performance
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.