Skip to content

Add Google Analytics to documentation#9

Merged
SkalskiP merged 1 commit into
mainfrom
add-google-analytics
Feb 1, 2023
Merged

Add Google Analytics to documentation#9
SkalskiP merged 1 commit into
mainfrom
add-google-analytics

Conversation

@capjamesg

Copy link
Copy Markdown
Contributor

Description

This PR adds Google Analytics to the Supervision documentation.

Type of change

  • Documentation update

How has this change been tested, please provide a testcase or example of how you tested the change?

This change was tested by ensuring that the requisite GA tag loaded successfully on the project documentation.

Any specific deployment considerations

N/A

Docs

N/A

@capjamesg capjamesg self-assigned this Feb 1, 2023
@SkalskiP SkalskiP merged commit b5e33c5 into main Feb 1, 2023
@capjamesg capjamesg deleted the add-google-analytics branch July 5, 2023 08:49
Borda added a commit that referenced this pull request Apr 14, 2026
[resolve #9] /review finding by qa-specialist (report: .temp/output-review-from-inference-rle-mask-2026-04-14.md):
"Consider adding a test with bytes that would fail UTF-8 decode to verify error handling..."

---
Co-authored-by: Claude Code <noreply@anthropic.com>
Borda added a commit that referenced this pull request Apr 15, 2026
Decorated functions now preserve __name__, __doc__, and __wrapped__
metadata, fixing introspection and documentation generation for all
annotators using ensure_cv2/pil_image_for_* decorators.

[resolve #9] Review comment by @copilot (PR #2214):
"The decorator wrappers in this file no longer use functools.wraps..."

---
Co-authored-by: Claude Code <noreply@anthropic.com>
Borda added a commit that referenced this pull request Apr 15, 2026
Decorated functions now preserve __name__, __doc__, and __wrapped__
metadata, fixing introspection and documentation generation for all
annotators using ensure_cv2/pil_image_for_* decorators.

[resolve #9] Review comment by @copilot (PR #2214):
"The decorator wrappers in this file no longer use functools.wraps..."

---
Co-authored-by: Claude Code <noreply@anthropic.com>
Borda added a commit to shaun0927/supervision that referenced this pull request Apr 17, 2026
- Add Raises: section listing all three ValueError cases (equality guard,
  parents escape, is_dir guard)
- Add Note: explaining path-containment validation and symlink rejection

[resolve roboflow#9] /review finding by doc-scribe (report: .temp/output-review-fix-coco-loader-path-containment-2026-04-17.md):
"Missing Raises: section — function raises ValueError in 3 distinct undocumented cases"

---
Co-authored-by: Claude Code <noreply@anthropic.com>
Borda added a commit that referenced this pull request Apr 22, 2026
- Rename TestResize → TestCompactMaskResize for naming consistency with
  other test classes in test_compact_mask.py
- Add test_join_true_true_junction_no_zero_run targeting the True/True
  boundary merge introduced in the prior fix
- Move _PARALLEL_THRESHOLD to module level alongside _L3_DENSITY_THRESHOLD
- Cap ThreadPoolExecutor max_workers=min(n, os.cpu_count() or 4) to avoid
  thread contention in library code
- Document identity-resize shallow-copy semantics in inline comment

[resolve #3] @Copilot (gh): "TestResize is a very generic class name..."
[resolve #9] /review by sw-engineer: "_PARALLEL_THRESHOLD constant buried in method body"
[resolve #10] /review by sw-engineer: "ThreadPoolExecutor() with no max_workers argument"
[resolve #11] /review by sw-engineer: "identity resize does shallow copy of list(self._rles)"
[resolve #12] /review by qa-specialist: "_rle_join_cols has no test for True/True junction"

---
Co-authored-by: Claude Code <noreply@anthropic.com>
Borda added a commit that referenced this pull request May 22, 2026
[resolve #9] /review finding by codex (report: .reports/review/2026-05-22T05-56-29Z/review-report.md)

---
Co-authored-by: Claude Code <noreply@anthropic.com>
Co-authored-by: OpenAI Codex <codex@openai.com>
Borda added a commit to madhavcodez/supervision that referenced this pull request May 22, 2026
Silently discarding the return value contradicted the just-described
chaining contract, leaving readers unsure whether the discard was
intentional. Use _, _ with an explanatory comment.

[resolve roboflow#9] /review finding by foundry:doc-scribe (report: .reports/review/2026-05-22T18-44-11Z/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.

2 participants