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.

Sign up for GitHub

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

Jump to bottom

refactorize postprocess #109

Merged
merged 14 commits into from
May 29, 2021
Merged

refactorize postprocess #109

merged 14 commits into from
May 29, 2021

Conversation

fcakyon
Copy link
Collaborator

@fcakyon fcakyon commented May 28, 2021 •
edited
Loading

  • specify postprocess type as --postprocess_type UNIONMERGE or --postprocess_type NMS to be applied over sliced predictions
  • specify postprocess match metric as --match_metric IOS for intersection over smaller area or --match_metric IOU for intersection over union
  • specify postprocess match threshold as --match_thresh 0.5
  • add --class_agnostic argument to ignore category ids of the predictions during postprocess (merging/nms)

fixes #108

@fcakyon fcakyon added the enhancement New feature or request label May 28, 2021
@fcakyon fcakyon self-assigned this May 28, 2021
@fcakyon fcakyon added this to the v0.4.3 milestone May 28, 2021
@fcakyon fcakyon merged commit 026bced into main May 29, 2021
@fcakyon fcakyon deleted the feature108 branch May 30, 2021 14:42
@fcakyon fcakyon changed the title simplify postprocess refactorize postprocess May 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@fcakyon fcakyon

Labels
enhancement New feature or request
Projects
None yet
Milestone
v0.4.3
Development

Successfully merging this pull request may close these issues.

postprocess functions are too complex
1 participant
@fcakyon