Skip to content

🐞 Bug Report, 🤩 Feature Request and ❓ Question issue templates#7

Merged
SkalskiP merged 8 commits into
mainfrom
feature/github-issue-templates
Jan 31, 2023
Merged

🐞 Bug Report, 🤩 Feature Request and ❓ Question issue templates#7
SkalskiP merged 8 commits into
mainfrom
feature/github-issue-templates

Conversation

@SkalskiP

Copy link
Copy Markdown
Collaborator

Description

  • 🐞 Bug Report, 🤩 Feature Request and ❓ Question issue templates added

@SkalskiP SkalskiP merged commit fcf30b7 into main Jan 31, 2023
@SkalskiP SkalskiP deleted the feature/github-issue-templates branch August 8, 2023 09:11
@likith1908 likith1908 mentioned this pull request Jul 13, 2024
1 task
Borda added a commit that referenced this pull request Apr 14, 2026
[resolve #7] /review finding by sw-engineer (report: .temp/output-review-from-inference-rle-mask-2026-04-14.md):
"rle_to_mask uses assert for input validation which can be disabled with python -O; raise ValueError instead..."

---
Co-authored-by: Claude Code <noreply@anthropic.com>
Borda added a commit that referenced this pull request Apr 15, 2026
- Move supervision/keypoint/ → src/supervision/keypoint/ so shims are
  included in built wheels (project uses src layout)
- Replace raw DeprecationWarning with warn_deprecated (SupervisionWarnings)
  in all three shim files for consistent warning behaviour
- Fix wrong import path (supervision.utils.deprecation → .internal)
  and remove unsupported stacklevel arg from warn_deprecated call

[resolve #7] Review comment by @copilot (PR #2214):
"This warning uses the standard DeprecationWarning..."
[resolve #8] Review comment by @copilot (PR #2214):
"These deprecation shim modules are added under the top-level supervision/..."

---
Co-authored-by: Claude Code <noreply@anthropic.com>
Borda added a commit that referenced this pull request Apr 15, 2026
- Move supervision/keypoint/ → src/supervision/keypoint/ so shims are
  included in built wheels (project uses src layout)
- Replace raw DeprecationWarning with warn_deprecated (SupervisionWarnings)
  in all three shim files for consistent warning behaviour
- Fix wrong import path (supervision.utils.deprecation → .internal)
  and remove unsupported stacklevel arg from warn_deprecated call

[resolve #7] Review comment by @copilot (PR #2214):
"This warning uses the standard DeprecationWarning..."
[resolve #8] Review comment by @copilot (PR #2214):
"These deprecation shim modules are added under the top-level supervision/..."

---
Co-authored-by: Claude Code <noreply@anthropic.com>
Borda added a commit to shaun0927/supervision that referenced this pull request Apr 17, 2026
…ocstrings

[resolve roboflow#7] /review finding by foundry:doc-scribe (report: .temp/output-review-fix-sinks-slice-list-values-2026-04-17.md)

---
Co-authored-by: Claude Code <noreply@anthropic.com>
Co-authored-by: OpenAI Codex <codex@openai.com>
Borda added a commit to shaun0927/supervision that referenced this pull request Apr 17, 2026
The smooth parameter docstring now describes the fallback to raw
polyline when fewer than 4 unique anchor points are available.

[resolve roboflow#7] /review finding by doc-scribe (report: smooth param docstring omits fallback)

---
Co-authored-by: Claude Code <noreply@anthropic.com>
Borda added a commit to shaun0927/supervision that referenced this pull request Apr 17, 2026
Add parametrized test for '.', '', 'subdir/..' — all resolve to the images
directory itself, exercising the equality guard that had zero coverage.

[resolve roboflow#7] /review finding by qa-specialist (report: .temp/output-review-fix-coco-loader-path-containment-2026-04-17.md):
"Branch 1 (equality guard) has zero test coverage"

---
Co-authored-by: Claude Code <noreply@anthropic.com>
Borda added a commit that referenced this pull request Apr 22, 2026
- Add test_resize_dispatch_uses_cv2_for_dense: verifies _resize_crop takes
  the cv2 fallback for a checkerboard mask that exceeds _L3_DENSITY_THRESHOLD
- Add TestResizeParallelPath with two tests covering N>=8 masks (triggers
  ThreadPoolExecutor path): correctness and shape checks

[resolve #7] /review finding by qa-specialist (report: .temp/output-review-rle-resize-2026-04-22.md):
"No test for density-based dispatch threshold (_L3_DENSITY_THRESHOLD)"

[resolve #8] /review finding by qa-specialist (report: .temp/output-review-rle-resize-2026-04-22.md):
"No test for resize() with N >= 8 (thread pool code path never exercised)"

---
Co-authored-by: Claude Code <noreply@anthropic.com>
Borda added a commit to satishkc7/supervision that referenced this pull request May 19, 2026
…be (report: .reports/review/2026-05-19T10-05-17Z/review-report.md): document silent-degrade contract for preserve_audio

- Clarify that missing/failing ffmpeg warns and continues rather than raising
- Add install hint for ffmpeg (apt/brew)
- Note that audio is truncated to match the processed video duration (-shortest)

---
Co-authored-by: Claude Code <noreply@anthropic.com>
Borda added a commit to madhavcodez/supervision that referenced this pull request May 22, 2026
…etections

Three additions:
- Assert cv2.imwrite return value in _tiny_detection_dataset helper so
  codec failures produce explicit errors rather than cryptic downstream ones
- Fix _tiny_detection_dataset xyxy shape for dets_per_image=0 (was (0,),
  now (0,4) via reshape — required by Detections validator)
- test_save_coco_annotations_annotation_image_id_references_correct_image:
  verifies every annotation.image_id references a known image id after
  chaining with a non-default starting_image_id
- test_save_coco_annotations_zero_annotation_images: pins behaviour for
  images with zero detections — image ids advance, annotation list empty

[resolve roboflow#6] /review finding by foundry:qa-specialist (report: .reports/review/2026-05-22T18-44-11Z/review-report.md):
[resolve roboflow#7] /review finding by foundry:qa-specialist (report: .reports/review/2026-05-22T18-44-11Z/review-report.md):
[resolve roboflow#10] @Copilot (gh comment #3290385653):

---
Co-authored-by: Claude Code <noreply@anthropic.com>
Borda added a commit to Ace3Z/supervision that referenced this pull request May 27, 2026
Function had no docstring. Document: CLASS_NAME_DATA_FIELD always present and
string-dtype in returned data dict (load-bearing invariant for the empty-path
fix), 6-tuple return order, and an example showing the dtype guarantee.

[resolve roboflow#7] /review finding by foundry:doc-scribe (report: .reports/review/2026-05-27T06-06-27Z/review-report.md)

---
Co-authored-by: Claude Code <noreply@anthropic.com>
Borda added a commit to kounelisagis/supervision that referenced this pull request Jun 8, 2026
- [resolve roboflow#5] oriented_box_non_max_suppression + oriented_box_non_max_merge:
  add Examples: doctest blocks (project convention for all public APIs).
  Challenge: evidence=VALID suggestion=VALID resolution=as-suggested
- [resolve roboflow#7] oriented_box_iou_batch: update summary/Returns from "Pairwise IoU"
  to overlap_metric-aware wording, matching box_iou_batch convention.
  Challenge: evidence=VALID suggestion=VALID resolution=as-suggested
- [resolve roboflow#8] oriented_box_iou_batch + public entry-points: add ndim/shape
  validation before reshape to catch flat (N,8) and wrong-ndim inputs early.
  Challenge: evidence=VALID suggestion=VALID resolution=as-suggested
- [resolve roboflow#9] oriented_box_non_max_merge: add assert 0 <= iou_threshold <= 1,
  matching the guard present on oriented_box_non_max_suppression and siblings.
  Challenge: evidence=VALID suggestion=VALID resolution=as-suggested

[resolve group] PR roboflow#2303 — items 5, 7, 8, 9

---
Co-authored-by: claude[bot] <209825114+claude[bot]@users.noreply.github.com>
Co-authored-by: OpenAI Codex <codex@openai.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant