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

Back up blob archives and fix saving blob columns #216

Merged
merged 14 commits into from
Sep 29, 2022

Conversation

yoda-vid
Copy link
Collaborator

We're splitting out the blob changes from the classifier branch (#199) here for modularity and to integrate them sooner.

  • Blob archives are backed up before overwriting
  • Fixed saving additional blob columns
  • Fixed blob accessors/setters
  • Fixed blob detections in the GUI with no selected channels or loaded blobs but none in the ROI
  • Fixed exporting ROIs
  • Reduced debug messages during blob detection

Change type hint to indicate that `imshow_multichannel` does not take a scalar as channel. Update `setup_channels` to use type hints.
…n set up

Accessing a blob column of None has unintentionally added an axis to the blobs. A column of None is given when the column is missing from the blob, so the blob column accessor now returns None or an empty array depending on the blobs dimensions if the column is None.
Check image value for None directly. Also, fix docstring to indicate that the attribute is an Image5d instance.
- Remove the channel from the shape before reversing the order
- Remove unused ROI to Image5d array call
Pass the channel as a sequence as required for the plotter.
Formatting blobs with additional columns has not added mappings for the new columns to their column indices, fixed here. The mapped columns are now saved instead of the loaded columns in case the mapping has changed.
Add a parameter that allows setting a mask for the first axis, defaulting to an index expression that gets all rows. Change specific column setters to pass args/kwargs to reduce redundancy.
Decrease potential noise from debugging messages during blob detection.
Avoid overwriting an existing blobs archive.
- Change from int to optional sequence of ints since this value is set up as a list from the CLI
- Change channel setup type hint to allow channel to be None
…he ROI

When extracting blobs during GUI detections, the blobs is an empty array rather than None, which led to an error when getting a discrete colormap based on the blob channels, fixed here.
Provide an error message and return immediately when no channels are selected during blob detection.
@yoda-vid yoda-vid added bug Something isn't working enhancement New feature or request labels Sep 29, 2022
@yoda-vid yoda-vid added this to the v1.6.0 milestone Sep 29, 2022
@yoda-vid yoda-vid merged commit 508cd42 into master Sep 29, 2022
@yoda-vid yoda-vid deleted the fix_save_blob_columns branch September 29, 2022 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant