-
Notifications
You must be signed in to change notification settings - Fork 114
Fix multi size video support #501
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
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Codecov Report
@@ Coverage Diff @@
## develop #501 +/- ##
===========================================
+ Coverage 64.94% 64.97% +0.02%
===========================================
Files 124 124
Lines 19669 19691 +22
===========================================
+ Hits 12775 12795 +20
- Misses 6894 6896 +2
Continue to review full report at Codecov.
|
- Lint/line length
dbbf443
to
d251fbb
Compare
arie-matsliah
added a commit
that referenced
this pull request
Mar 8, 2021
* Merge main back into develop (#491) * Prep for master->main (#475) Co-authored-by: Arie Matsliah <arie.matsliah@gmail.com> * Release version 1.1.0 * Remove appveyor Co-authored-by: Talmo Pereira <talmo@princeton.edu> Co-authored-by: Arie Matsliah <arie.matsliah@gmail.com> * Merge main to develop post release (#492) * Prep for master->main (#475) Co-authored-by: Arie Matsliah <arie.matsliah@gmail.com> * Release version 1.1.0 * Remove appveyor * Test docs subfolders * Test keep_files Co-authored-by: Talmo Pereira <talmo@princeton.edu> Co-authored-by: Arie Matsliah <arie.matsliah@gmail.com> * Pulling finished work from #484 into a separate PR (#493) * Pulling finished work into a separate PR * fix test * fix * Simplify + tweak background * Update docs (#495) * Isolated fix for issue #480 (#498) * Add support for specifying the minimum score in PAF matching (#502) Effective at filtering out explicitly spurious matches when no better match is available * Fix post-training errors (#503) - Catch filesystem busy error when deleting viz dir - Don't throw an error after training with no inference target * Revert "Pulling finished work from #484 into a separate PR" (#504) * Minor fixes and enhacements (#505) - Add skeleton.find_neighbors convenience graph op - Add Instance.fill_missing to initialize new nodes in the skeleton - Also add numpy() and from_pointsarray() aliases to PredictedInstance to prevent crashes from using the inherited versions - Initialize missing instance nodes when plotting user instances - Fix serialization after loading MediaVideo from high level API - Fix single instance visualization during training - Add --open-in-gui CLI option to sleap-track - Clearer error message when running sleap-track without args * [WIP] Fix multi size video support (#501) * wip * cleanup * Add `max_height_and_width` property to `VideoReader` * Fix SizeMatcher instantiation without specified targets - Lint/line length * Provider now recreates the data pipeline with new providers * Vectorize padding test for SizeMatcher * Do not require instances key if not available from provider * Size matching * Size matching fix * Fix shapes in output dataset * lint Co-authored-by: Talmo Pereira <talmo@princeton.edu> * Bump version for release (#506) Co-authored-by: Talmo Pereira <talmo@princeton.edu> Co-authored-by: Arie Matsliah <arie.matsliah@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
[describe your changes here]
Types of changes
Does this address any currently open issues?
[list open issues here]