Skip to content

chore: remove EdgeTAM encoder profiler - #28

Closed
PABannier wants to merge 1 commit into
cleanup/remove-test-apifrom
cleanup/remove-edgetam-profiler
Closed

PABannier wants to merge 1 commit into
cleanup/remove-test-apifrom
cleanup/remove-edgetam-profiler

Conversation

@PABannier

Copy link
Copy Markdown
Owner

sam3_profile_edgetam_encode is 437 lines in sam3.cpp plus a public declaration, a 96-line example, a CMake target and a release-packaging entry. Its one caller is examples/profile_edgetam.cpp. It rebuilds the RepViT stem, stages and FPN neck as separate sub-graphs only to print per-stage timings. sam3_benchmark --filter edgetam already reports load / encode / per-frame latency for every backend, and a one-off stage breakdown can be redone from git history.

Removed:

  • sam3_profile_edgetam_encode and its private helpers (sam3_profile_*) in sam3.cpp
  • the "Profiling" section in sam3.h
  • examples/profile_edgetam.cpp and its target in examples/CMakeLists.txt
  • sam3_profile_edgetam from the binaries packaged by .github/workflows/release.yml (macOS/Linux and Windows)

Stacked on #20 because both PRs delete adjacent sections of sam3.h. Merge #20 first; GitHub will retarget this PR to main.

Verification: a parity tool runs encode, PVS point + box, and a 3-frame visual track, then compares the output byte for byte with origin/main:

PARITY OK   edgetam_f16 gpu0
PARITY OK   edgetam_f16 gpu1
PARITY OK   sam2.1_hiera_tiny_q8_0 gpu0
PARITY OK   sam2.1_hiera_tiny_q8_0 gpu1

The full Release build (library + all examples) has no errors, and git grep profile_edgetam is empty.

git diff --shortstat: 5 files changed, 2 insertions(+), 561 deletions(-).

Part of an over-engineering cleanup series.

🤖 Generated with Claude Code

https://claude.ai/code/session_01M2qLmQ9ag49hT61V6i9qAq

sam3_profile_edgetam_encode (437 lines) and its profile_edgetam example
re-built the RepViT/FPN graphs stage by stage just to print timings.
sam3_benchmark already reports per-model, per-backend latency. Drop
the function, its public declaration, the example target, and the
release packaging entry.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01M2qLmQ9ag49hT61V6i9qAq
@PABannier PABannier closed this Sep 16, 2026
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