evo2 SAE feature-explorer dashboard#1604
Draft
polinabinder1 wants to merge 5 commits into
Draft
Conversation
…ctor) WIP safety checkpoint. Live Evo2-1B + C13 SAE backend (steering_server.py: /health, /features, /annotate, /generate) + a 3-tab React app (Feature atlas, Generative steering, Sequence inspector). Multi-feature additive clamp-after-prompt steering; by-name feature picker; editable phylo tag. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
…ering = plain black DNA (no colormap); no-clamp generation + opt-in baseline; editable phylo tag; by-name multi-feature picker Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
…er serve The dashboard carried its own 686-line steering_server.py that re-implemented Evo2 + SAE loading, the forward pass, and the /health,/features,/annotate,/generate endpoints — all of which now live in the standalone evo2_sae_infer engine. Delete it; the front-end already reaches :8001 via the Vite /api proxy, so it now consumes `evo2_sae_infer serve` (whose endpoint set + response fields are a superset). Add a README documenting the front-end / backend split. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Move the 164-line static `styles` object into src/styles.js. App.jsx drops 1634 -> 1470 lines and styling is isolated. Pure move — module-level static object used as styles.*, no behavior change. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
zoomToPoint + resetViewport (two fly-to / zoom-out useCallback animations) and the easeOutQuart/easeInOutCubic/easeInOutQuad helpers they used were declared but never referenced anywhere (each occurs only at its own declaration). Removing them drops App.jsx 1470 -> 1352. No behavior change. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Contributor
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
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
1 participant
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.
Summary
React/Vite feature-explorer dashboard for Evo2 SAE features — three tabs: feature
atlas, sequence inspector, and generative steering. Front-end only; its
backend is the standalone
evo2_sae_infer serveengine (#1603).That backend is a runtime dependency (a separate process the dashboard calls via the
Vite
/api→:8001proxy), not a code dependency — so this PR's diff is purely thefeature_explorer/front-end and the two PRs are independent (merge in any order).Refactors included here
steering_server.py— the dashboard now uses theshared
evo2_sae_inferengine (evo2 SAE inference engine + CLI (encode / batch / serve) #1603) instead of re-implementing model/SAE loading + endpoints.stylesobject out ofApp.jsxintosrc/styles.js.zoomToPoint/resetViewport+ their easinghelpers, all unreferenced):
App.jsx1,634 → 1,352.feature_explorer/net: 4,905 → 4,123 lines.Note
The React build was not verified in the authoring environment — please run
npm run build+ a visual check (and anevo2_sae_infer servesmoke) before marking ready.🤖 Generated with Claude Code