Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow generate_deepcell_input to accept other img_sub_folder values (including None) for non-MIBItiff inputs #471

Merged
merged 3 commits into from
Oct 1, 2021

Conversation

alex-l-kong
Copy link
Contributor

What is the purpose of this PR?

Closes #470. A bug in generate_deepcell_input defaulted the img_sub_folder value passed into load_imgs_for_tree to always be 'TIFs'. We need it to accept the img_sub_folder value passed into generate_deepcell_input instead.

How did you implement your changes

In addition to changing this, we also change the default value for img_sub_folder of generate_deepcell_input to be None instead of 'TIFs'.

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@alex-l-kong alex-l-kong self-assigned this Sep 30, 2021
Copy link
Member

@ngreenwald ngreenwald left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Two small changes

ark/utils/data_utils.py Outdated Show resolved Hide resolved
templates/Segment_Image_Data.ipynb Outdated Show resolved Hide resolved
@ngreenwald ngreenwald merged commit 118b8e6 into master Oct 1, 2021
@ngreenwald ngreenwald deleted the deepcell_path_patch branch October 1, 2021 02:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

generate_deepcell_input in data_utils does not allows for img_sub_folder to be None or empty str
2 participants