Skip to content

[FOEPD-3392] Implement image embedding caching in IndexedDB#7223

Draft
yrahal wants to merge 1 commit intoyoucef/sam2_2from
youcef/sam2_3
Draft

[FOEPD-3392] Implement image embedding caching in IndexedDB#7223
yrahal wants to merge 1 commit intoyoucef/sam2_2from
youcef/sam2_3

Conversation

@yrahal
Copy link
Contributor

@yrahal yrahal commented Mar 20, 2026

What changes are proposed in this pull request?

Caches SAM2 encoder embeddings in IndexedDB with in-memory LRU eviction so repeated prompts on the same image skip encoding and run decoder only (~150ms vs ~1-3s)

How is this patch tested? If it is not, please explain why.

  • Unit tests
  • In-app (local uncommitted UI changes for testing)

Release Notes

Is this a user-facing change that should be mentioned in the release notes?

  • No. You can skip the rest of this section.
  • Yes. Give a description of this change to be included in the release
    notes for FiftyOne users.

(Details in 1-2 sentences. You can just refer to another PR with a description
if this PR is part of a larger change.)

What areas of FiftyOne does this PR affect?

  • App: FiftyOne application changes
  • Build: Build and test infrastructure changes
  • Core: Core fiftyone Python library changes
  • Documentation: FiftyOne documentation changes
  • Other

Caches SAM2 encoder embeddings in IndexedDB with in-memory LRU eviction so repeated prompts on the same image skip encoding and run decoder only (~150ms vs ~1-3s)
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 20, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: b688c7ce-3cc3-49ac-b240-8e5f5e22f216

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch youcef/sam2_3

Comment @coderabbitai help to get the list of available commands and usage tips.

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