-
Notifications
You must be signed in to change notification settings - Fork 63
Export bboxes dataset as VIA-tracks file #580
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
Open
sfmig
wants to merge
75
commits into
main
Choose a base branch
from
smg/save-via-tracks-from-pr497
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
75 commits
Select commit
Hold shift + click to select a range
3b72492
Add files via upload
harsh-bhanushali-05 9ae5b89
Add support for exporting bboxes in VIA-tracks
harsh-bhanushali-05 d6ab206
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 5fbe43c
Bug fixes
harsh-bhanushali-05 be489d2
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 7a74ce7
More big fixes
harsh-bhanushali-05 d51f08b
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 0a31390
big fix
harsh-bhanushali-05 a00d6da
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 3ff9ab6
Pre commit changes
harsh-bhanushali-05 a93a1cc
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] da2aa29
Pre-commit error changes
harsh-bhanushali-05 37dd94b
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 5fe2fca
fix
harsh-bhanushali-05 fa3c97d
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 45c7e16
fix.
harsh-bhanushali-05 e8a8965
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] aa9d4b5
Fixing code to resolve CI testcases
harsh-bhanushali-05 ac25ba4
Corrected the export format.
harsh-bhanushali-05 830e5ad
Updated testcases
harsh-bhanushali-05 eab3584
Updated the testcase
harsh-bhanushali-05 f5ebd0d
Replace logging with loguru
harsh-bhanushali-05 5d7a72d
Updated the logging
harsh-bhanushali-05 4801ec8
Rename file and small edits
sfmig 227aab8
Export confidence optionally and pad with max digits plus one
sfmig 637756e
Add preliminary tests
sfmig 3d791b3
Parametrize tests
sfmig 2a9d322
Add test_to_via_tracks_file_invalid_dataset
sfmig c62f3a4
Add invalid file test
sfmig 5771d10
Add map to individual track ID and bboxes dataset validator
sfmig 4da9037
Rename function to validate poses dataset
sfmig 9e8620b
Add tests for mapping
sfmig bbbc413
Add mapping function. Factor out full csv writing
sfmig 626c94f
Extend tests
sfmig 03d5d5f
Remove renamed module
sfmig 1a3bb6c
Get frame numbers from time coordinates (rather than from array shape)
sfmig 9f8a65a
Move csv_writer call inside row writing function. Change max_digits t…
sfmig 61afc3d
Use string literal for csv writer type annotation - see https://til.c…
sfmig e8405cc
Review fn signatures. Add image file suffix
sfmig f611eb4
Factor out common io utils and add explicit tests
sfmig 65a5ffe
Change regex to non-greedy to fix sonarcloud issue (will check after …
sfmig bba4a07
Try to fix sonarcloud issue
sfmig 547ef88
Replace regexp by str methods to make sonarcloud happy
sfmig 492fd77
Add image_file_suffix to test
sfmig 392e2ff
Add image size to test
sfmig f57411a
Test internal functions only for cases that are relevant
sfmig 217bdd7
Define image filename template
sfmig 8d2dba3
Reorganise and rename some functions. Add tests for get_image_filenam…
sfmig a62e9a1
Reorder tests
sfmig 206ea60
Review docstrings and small edits. Factor out _get_track_id_from_indi…
sfmig db3e137
Expand test_to_via_tracks_file_valid_dataset
sfmig 3ba25f6
Separate test of to_via_tracks_file when using different values of ex…
sfmig 7b097f7
Separate image filename and confidence tests. Write confidence tests
sfmig 038b012
Fix quotes for loadable in VIA (WIP, test fix pending)
sfmig db88119
Loadable in VIA
sfmig 767c2cf
Replace with json approach
sfmig 1524a22
Fix json serialisation issue
sfmig 8db30ea
Add preliminary test for double quotes
sfmig 842bbbc
Fix frame_ being interpreted as a cross-reference by sphinx
sfmig 21d11f8
Small edits to test
sfmig c4c1fa2
Allow user to set padding for frame number
sfmig d83d7b9
Rename variables. Add helper function for tests
sfmig b2c645a
Add region id and count id
sfmig 8e18070
Check two lines literally
sfmig b652dd3
Combine image filename argus to _write_single_row into one
sfmig 0a43e1c
Add test to check datasets are recoverable
sfmig 35396e0
Add tests for region and count ID
sfmig a628c67
Change default extract_track_id_from_individuals to True
sfmig 4d74049
Small edits
sfmig 3eb9c75
Fixes for API reference
sfmig 0ad84b2
Consistency in naming the file
sfmig 4f66088
Add reference
sfmig 76a0eee
Update guide
sfmig 61c5ecd
Add reference
sfmig 3c2dbc7
Update guide
sfmig File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Now that we have the built-in support for saving to VIA, I wonder if it's worth putting this "alternative" custom csv option under a dropdown, in the spirit of reducing the page length