Skip to content

Commit

Permalink
adjust mask suffix input
Browse files Browse the repository at this point in the history
  • Loading branch information
camisowers committed Oct 7, 2024
1 parent 9dab65b commit f168dfb
Show file tree
Hide file tree
Showing 2 changed files with 48 additions and 5 deletions.
16 changes: 11 additions & 5 deletions src/ark/segmentation/marker_quantification.py
Original file line number Diff line number Diff line change
Expand Up @@ -453,9 +453,9 @@ def create_marker_count_matrices(segmentation_labels, image_data, nuclear_counts


def generate_cell_table(segmentation_dir, tiff_dir, img_sub_folder="TIFs",
is_mibitiff=False, fovs=None,
extraction='total_intensity', nuclear_counts=False,
fast_extraction=False, mask_types=['whole_cell'], **kwargs):
is_mibitiff=False, fovs=None, extraction='total_intensity',
nuclear_counts=False, fast_extraction=False, mask_types=['whole_cell'],
add_underscore=True, **kwargs):
"""This function takes the segmented data and computes the expression matrices batch-wise
while also validating inputs
Expand All @@ -480,6 +480,8 @@ def generate_cell_table(segmentation_dir, tiff_dir, img_sub_folder="TIFs",
if set, skips the custom regionprops and expensive base regionprops extraction steps
mask_types (list):
list of masks to extract data for, defaults to ['whole_cell']
add_underscore (str):
whether to add '_' before the mask type suffix in file names, defaults to True
**kwargs:
arbitrary keyword arguments for signal and regionprops extraction
Expand Down Expand Up @@ -529,13 +531,17 @@ def generate_cell_table(segmentation_dir, tiff_dir, img_sub_folder="TIFs",
fovs=[fov_name])

for mask_type in mask_types:
if mask_type is None:
mask_type, mask_suff = 'cell_mask', None
else:
mask_suff = '_' + mask_type if add_underscore else mask_type
# load the segmentation labels in
fov_mask_name = fov_name + '_' + mask_type + ".tiff"
fov_mask_name = fov_name + mask_suff + ".tiff" if mask_suff else fov_name + ".tiff"
current_labels_cell = load_utils.load_imgs_from_dir(data_dir=segmentation_dir,
files=[fov_mask_name],
xr_dim_name='compartments',
xr_channel_names=[mask_type],
trim_suffix='_' + mask_type)
trim_suffix=mask_suff)

compartments = ['whole_cell']
segmentation_labels = current_labels_cell.values
Expand Down
37 changes: 37 additions & 0 deletions tests/segmentation/marker_quantification_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -751,6 +751,43 @@ def test_generate_cell_table_tree_loading():
nuc_cell_table_cols=arcsinh_data_nuc.columns.values
)

# tests for different mask suffixes
for fov in range(cell_masks_40.shape[0]):
fov_whole_cell = cell_masks_40[fov, :, :, 0]
fov_nuclear = cell_masks_40[fov, :, :, 1]
image_utils.save_image(os.path.join(temp_dir, 'fov%d.tiff' % fov),
fov_whole_cell)
image_utils.save_image(os.path.join(temp_dir, 'fov%dtest.tiff' % fov),
fov_nuclear)

for fov in range(cell_masks_20.shape[0]):
fov_whole_cell = cell_masks_20[fov, :, :, 0]
fov_nuclear = cell_masks_20[fov, :, :, 1]
image_utils.save_image(
os.path.join(temp_dir, 'fov%d.tiff' % (fov + fov_size_split)),
fov_whole_cell
)
image_utils.save_image(
os.path.join(temp_dir, 'fov%dtest.tiff' % (fov + fov_size_split)),
fov_nuclear
)

# generate sample norm and arcsinh data for all fovs with no suffix
norm_data_all_fov, arcsinh_data_all_fov = marker_quantification.generate_cell_table(
segmentation_dir=temp_dir, tiff_dir=tiff_dir,
img_sub_folder=img_sub_folder, is_mibitiff=False, fovs=None, mask_types=[None])

assert norm_data_all_fov.shape[0] > 0 and norm_data_all_fov.shape[1] > 0
assert arcsinh_data_all_fov.shape[0] > 0 and arcsinh_data_all_fov.shape[1] > 0

# generate sample norm and arcsinh data for all fovs with 'test' suffix
norm_data_all_fov, arcsinh_data_all_fov = marker_quantification.generate_cell_table(
segmentation_dir=temp_dir, tiff_dir=tiff_dir, img_sub_folder=img_sub_folder,
is_mibitiff=False, fovs=None, mask_types=['test'], add_underscore=False)

assert norm_data_all_fov.shape[0] > 0 and norm_data_all_fov.shape[1] > 0
assert arcsinh_data_all_fov.shape[0] > 0 and arcsinh_data_all_fov.shape[1] > 0


# TODO: consider removing since MIBItiffs are being phased out
def test_generate_cell_table_mibitiff_loading():
Expand Down

0 comments on commit f168dfb

Please sign in to comment.