Skip to content

Pipelines with input detections#590

Merged
BryonLewis merged 19 commits into
mainfrom
pipeline-existing-detections
Mar 6, 2021
Merged

Pipelines with input detections#590
BryonLewis merged 19 commits into
mainfrom
pipeline-existing-detections

Conversation

@BryonLewis
Copy link
Copy Markdown
Collaborator

@BryonLewis BryonLewis commented Feb 22, 2021

Adding in the capability to input detections/tracks into pipelines with the uility_ name.

  • generate_ pipelines are renamed to utility_ and require user input
  • Updated the generate category to utility ("utilities") to reflect the new structure
  • add segmentation watershed and track user selections are the two pipelines that expect user input and will properly generate detections from that. add segementation watershed will take existing detections and generate polygons for them. track user selections will take a single detection and create a track for it.
  • Added in some templating for passing tracks/detections into pipelines
  • Right now ALL utility pipelines will pass the detections in, the pipelines that don't accept it will ignore it.
  • Passing in an empty file will cause the pipeline to fail like it should
  • There are checks to prevent passing in an empty pipeline on web version

@BryonLewis BryonLewis linked an issue Feb 22, 2021 that may be closed by this pull request
@BryonLewis BryonLewis force-pushed the pipeline-existing-detections branch from 09a48fe to 69156f5 Compare February 25, 2021 13:40
@BryonLewis BryonLewis force-pushed the pipeline-existing-detections branch from faa94d6 to 95efbcf Compare March 1, 2021 00:12
@BryonLewis BryonLewis changed the title [WIP] - Pipelines with input detections Pipelines with input detections Mar 4, 2021
@BryonLewis BryonLewis marked this pull request as ready for review March 4, 2021 16:11
Copy link
Copy Markdown
Contributor

@subdavis subdavis left a comment

Choose a reason for hiding this comment

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

Looks great. I like the error for no detections.

@BryonLewis BryonLewis merged commit d9d67a8 into main Mar 6, 2021
@subdavis subdavis deleted the pipeline-existing-detections branch March 15, 2021 16:02
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.

Also feed in existing detections (if they exist) into processing pipelines

2 participants