Skip to content
This repository has been archived by the owner on Jan 3, 2024. It is now read-only.

Revert cell detection multiprocessing to old version #81

Closed
dstansby opened this issue Feb 28, 2023 · 1 comment
Closed

Revert cell detection multiprocessing to old version #81

dstansby opened this issue Feb 28, 2023 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@dstansby
Copy link
Member

dstansby commented Feb 28, 2023

As a stopgap solution to #73 we (myself, @alessandrofelder and @adamltyson) decided to revert the multiprocessing logic in the cell detection to an older version that did not have the current memory issues, and put out a release.

I plan to:

  1. Reproduce the memory build up locally on my laptop (done in Use random arrays for memory testing #82)
  2. Revert to an older version of the multiprocessing logic
  3. Make sure that the memory build up isn't an issue with the older logic
@dstansby dstansby added the enhancement New feature or request label Feb 28, 2023
@dstansby dstansby self-assigned this Feb 28, 2023
@dstansby
Copy link
Member Author

Decided not to go forwards with this

@dstansby dstansby closed this as not planned Won't fix, can't repro, duplicate, stale Mar 30, 2023
willGraham01 pushed a commit that referenced this issue Aug 24, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant