Closed camisowers closed 3 months ago
Check out this pull request on
See visual diffs & provide feedback on Jupyter Notebooks.
Powered by ReviewNB
Code looks good with, just need to update the example_dataset and can then merge with main.
@camisowers Dataset is updated, this is good to go.
If you haven't already, please read through our contributing guidelines before opening your PR
What is the purpose of this PR?
Closes #1125.
nuclear_counts=True
, the nuclear counts are calculated in a loop for each mask (large/unnecessary computation burden for many ezseg masks). The cell table creates it's own set of rows for the nuclear masks according to the ezseg logic, while also having a second set of columns suffixed with _nuclear using traditional cell table logic. We want to remove the duplicated nuclear expression values, as well as saving the computational effort.How did you implement your changes
mask_types
as an optional argument togenerate_cell_table()
detailing which masks to extract expression data for; the default is set to ['whole_cell'].Remaining issues
Add manual mask name specification into the ez segmenter notebook. @bryjcannon