Conversation
add file_col to generator call
add convnext framework
243 nan detections
Fix for #247 collate function not handling broken images correctly
…ication_empty change or to and and add else for empty not none check
hide internal utils from api
add station and camera in bfm, add unittests
…nput_checks add unit tests, check inputs for classification
…checks add detection fnf handling and unittests
… error Agent-Logs-Url: https://github.com/conservationtechlab/animl-py/sessions/fd0f522d-2312-44d4-8650-05a866237219 Co-authored-by: tkswanson <4371698+tkswanson@users.noreply.github.com>
Agent-Logs-Url: https://github.com/conservationtechlab/animl-py/sessions/4615a4dd-240f-43f2-ae46-2b98ef05132c Co-authored-by: tkswanson <4371698+tkswanson@users.noreply.github.com>
…omprehensive-unittest Add unit tests for pipeline.py + fix broken export.py import
…-for-visualization Add unit tests for visualization.py
Agent-Logs-Url: https://github.com/conservationtechlab/animl-py/sessions/9e5f24d7-f799-450b-b3b8-43ab4974d15d Co-authored-by: tkswanson <4371698+tkswanson@users.noreply.github.com>
Agent-Logs-Url: https://github.com/conservationtechlab/animl-py/sessions/9e5f24d7-f799-450b-b3b8-43ab4974d15d Co-authored-by: tkswanson <4371698+tkswanson@users.noreply.github.com>
…files-visualization-unittest Update tests to match current dev branch API
Agent-Logs-Url: https://github.com/conservationtechlab/animl-py/sessions/dea56731-27e0-4db7-a76b-79789fb5358d Co-authored-by: tkswanson <4371698+tkswanson@users.noreply.github.com>
…e-for-export-functions Add unit tests for animl/export.py
231 animlr check
Agent-Logs-Url: https://github.com/conservationtechlab/animl-py/sessions/e39dd8c7-1dce-45ff-8b61-605739ccb918 Co-authored-by: tkswanson <4371698+tkswanson@users.noreply.github.com>
…r-distance-module Add comprehensive unit tests for `animl/reid/`
matobler
reviewed
Apr 30, 2026
Collaborator
matobler
left a comment
matobler
left a comment
There was a problem hiding this comment.
Only a few minor comments.
| 'frame': frame['frame'], | ||
| 'max_detection_conf': frame['max_detection_conf'], | ||
| 'category': 0, 'conf': None, 'bbox_x': None, | ||
| 'category': None, 'conf': None, 'bbox_x': None, |
Collaborator
There was a problem hiding this comment.
Are we sure this change from category 0 to category None won't break things. MD uses 0 as Empty. What is the reason for this chagne?
Member
Author
There was a problem hiding this comment.
So that it can handle non-MD detectors correctly. I changed the way md is loaded and parsed to account for this change
|
|
||
| def active_times(manifest_dir, | ||
| depth: int = 1, | ||
| camera_depth: int = 0, |
Collaborator
There was a problem hiding this comment.
if we get the camera name during bulid_manifest can we just use that? Give a camera columns with default being the one in generated?
| timestamp_col (str): column in manifest to use for datetime information, defaults to "datetime" | ||
| recursive (bool): recursively search thhrough all child directories | ||
| offset (int): add timezone offset in hours to datetime column | ||
| data_timezone (str): timezone to apply to datetime column |
Collaborator
There was a problem hiding this comment.
with the new build_manifest timezone option the timezone shoudl be correct at this point.
| files["filemodifydate"] = files["filepath"].apply(lambda x: datetime.fromtimestamp(Path(x).stat().st_mtime).strftime('%Y-%m-%d %H:%M:%S')) | ||
|
|
||
| files = manifest_dir | ||
| # from scratch |
Collaborator
There was a problem hiding this comment.
do we need a "from scratch" option or can we just make users run build_file_manifest first?
…nifest Co-authored-by: Copilot <copilot@github.com>
Co-authored-by: Copilot <copilot@github.com>
Updated project description to be more descriptive.
…tion add count to sngle_classification
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
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.
No description provided.