Skip to content

Conversation

NetZissou
Copy link
Collaborator

@NetZissou NetZissou added documentation Improvements or additions to documentation enhancement New feature or request labels Jul 21, 2025
@NetZissou NetZissou marked this pull request as draft July 21, 2025 13:50
@NetZissou NetZissou marked this pull request as ready for review August 12, 2025 13:59
@NetZissou NetZissou requested a review from egrace479 August 12, 2025 13:59
…ated authors list; updated project URL sections
@egrace479 egrace479 requested a review from thompsonmj August 27, 2025 17:23
Copy link
Member

@egrace479 egrace479 left a comment

Choose a reason for hiding this comment

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

A few notes and questions

Homepage = "https://github.com/Imageomics/hpc-inference"
Documentation = "https://imageomics.github.io/hpc-inference/"
Repository = "https://github.com/Imageomics/hpc-inference"
Changelog = "https://github.com/Imageomics/hpc-inference/blob/main/CHANGELOG.md"
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Changelog = "https://github.com/Imageomics/hpc-inference/blob/main/CHANGELOG.md"


### Key Features

- **MegaDetector Support**: Optimized for Microsoft's MegaDetector models (specifically designed for wildlife camera trap images)
Copy link
Member

Choose a reason for hiding this comment

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

Is it that it's specifically designed to be use with Pytorch-Wildlife (https://github.com/microsoft/CameraTraps)?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I would compare PytorchWildlife to pybioclip. It provides a convenient Python interface for running inference with Megadetector, but it does not address I/O bottlenecks and is not optimized for scaling or for efficiently handling large inference tasks.

The Megadetector model weight files can be downloaded directly from the model zoo. hpc-inference interacts with these .pt model files directly.

Copy link
Member

Choose a reason for hiding this comment

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

But those come from Megadector?

The issue is specifying specifically what's used, but not misattributing it as "Microsoft's" Megadetector. Earlier code is licensed to Microsoft (see note in repo).

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Got it! We’re using Pytorch-Wildlife-model-weights, which were prepared by the PytorchWildLife team. The correct attribution should therefore go to PytorchWildlife, not Microsoft’s Megadetector.

Copy link
Member

Choose a reason for hiding this comment

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

I'd just add it to the citation further up (since you cite Megadetector with both already). Also, change:

Suggested change
- **MegaDetector Support**: Optimized for Microsoft's MegaDetector models (specifically designed for wildlife camera trap images)
- **MegaDetector Support**: Optimized for [Pytorch-Wildlife-model-weights](https://zenodo.org/records/15398270), specifically, the Megadetector models designed for wildlife camera trap images.

Comment on lines +169 to +173
- JPEG (.jpg, .jpeg)
- PNG (.png)
- BMP (.bmp)
- TIFF (.tif, .tiff)
- WebP (.webp)
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- JPEG (.jpg, .jpeg)
- PNG (.png)
- BMP (.bmp)
- TIFF (.tif, .tiff)
- WebP (.webp)
- JPEG (`.jpg`, `.jpeg`)
- PNG (`.png`)
- BMP (`.bmp`)
- TIFF (`.tif`, `.tiff`)
- WebP (`.webp`)


Configuration files provide better reproducibility and easier parameter management.

You can find documented template for config specification in `configs/animal_detection`
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
You can find documented template for config specification in `configs/animal_detection`
You can find documented templates for config specification in `configs/animal_detection`.

'class_name': 'animal'}]
```

Megadetector detected one animal with high confident 0.8981. Let's use the utility function to plot the detection box on top of the original image to validate the detection result:
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Megadetector detected one animal with high confident 0.8981. Let's use the utility function to plot the detection box on top of the original image to validate the detection result:
Megadetector detected one animal with high confidence (0.8981). Let's use the utility function to plot the detection box on top of the original image to validate the detection result:

title=f"Detections for {image_uuid}"
)
```
![Original vs Processed Beetle Images](imgs/detection_box.png)
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
![Original vs Processed Beetle Images](imgs/detection_box.png)
![Image 00170382-1e3d-4d46-9243-ac78136922ba.png with bounding box overlay for detection, original image from](imgs/detection_box.png)

Add in source for original image. May want to put in a caption or something.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Animal Detection Face Detection
2 participants