fix: avoid type-mutation bug in display_tp_fp_fn and normalize image_ids

[botbikamordehai2-sketch]

What

The display_tp_fp_fn method allowed image_ids to be a mixed list of ints and strings (e.g., from user input). The .get() call with a string key on an int-keyed dict silently fails (returns None) and causes some images to be skipped with a warning. Moreover, the method mutates the image_ids list when == ["all"] but does not protect against subsequent mutation of self.cocoGt.imgs during iteration.

Fix

1. Convert all non-"all" image_ids to int via [int(i) for i in image_ids] to ensure type consistency with the keys of self.cocoGt.imgs.
2. Use image_id not in self.cocoGt.imgs for existence check instead of .get() to avoid silent miss.
3. Iterate over a copy (already a list) to avoid mutation during iteration.

Closes #80

[Borda]

@botbikamordehai2-sketch I am very sure it does not close the mentioned #80 by far
pls update your PR describtion 🐰

[Copilot]

Pull request overview

This PR hardens PreviewResults.display_tp_fp_fn against mixed-type image_ids inputs by normalizing IDs and using an explicit existence check so valid images aren’t skipped due to key-type mismatches.

Changes:

• Normalize non-["all"] image_ids values to int before iterating.
• Replace dict.get(...) is None with image_id not in self.cocoGt.imgs for existence checking.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.