cytomining / pycytominer

Python package for processing image-based profiling data
https://pycytominer.readthedocs.io
BSD 3-Clause "New" or "Revised" License
76 stars 34 forks source link

Locate other performance bottlenecks, document, and add testing constraints #227

Open gwaybio opened 2 years ago

gwaybio commented 2 years ago

Discussed in https://github.com/cytomining/pycytominer/issues/195#issuecomment-1217265110

Add Memory/Time Testing Constraints for .merge_single_cells() (or others! 🙂 ) (Something like pytest-memray or similar): automate targeted performance tests to ensure operations within this project are usable by others as appropriate. Maybe this one spot was an outlier, but if there are other performance-heavy functions we could expand there as well.

jenna-tomkinson commented 1 year ago

@d33bs Is this issue resolved?

d33bs commented 1 year ago

@jenna-tomkinson - thanks, this is not yet resolved. In order for this to be resolved, I think the following steps would be needed (at a minimum):