Add Google Analytics to documentation#9
Merged
Merged
Conversation
SkalskiP
approved these changes
Feb 1, 2023
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>
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.
Description
This PR adds Google Analytics to the Supervision documentation.
Type of change
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