chore(catalog): remove 13 interactive-first specs#8645
Merged
Conversation
These specs' core value lies in click/hover/brush/animation behavior (drilldown navigation, brushing & linking, play controls, animated transitions) that anyplot's static renders cannot capture. Static implementations could only fake the interaction differently per library, and the static fallback always collapses into an existing spec (e.g. one drill level of bar-drilldown = bar-basic, one frame of bar-race-animated = bar-horizontal). None of them had JS implementations yet; all sat at 5-10 of 15 libraries. Removed specs: - bar-drilldown, pie-drilldown, map-drilldown-geographic - pie-portfolio-interactive, hierarchy-toggle-view - scatter-matrix-interactive, scatter-brush-zoom, linked-views-selection - dashboard-synchronized-crosshair, line-navigator - bar-race-animated, scatter-animated-controls, map-animated-temporal Postgres entries are cleaned up automatically by sync_to_postgres on push to main. GCS preview images will be removed separately after merge. Co-Authored-By: Claude Fable 5 <noreply@anthropic.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.
Summary
Removes 13 specs whose core value is interactivity (click drilldown, brushing & linking, hover sync, play controls, animation) that anyplot's static renders fundamentally cannot capture.
Why
bar-drilldownrenders a multi-panel exploded view), breaking cross-library comparability.bar-drilldown=bar-basic, one frame ofbar-race-animated= sortedbar-horizontal, one frame ofscatter-animated-controls=bubble-basic, static SPLOM =scatter-matrix, treemap/sunburst toggle =treemap-basic+sunburst-basic, etc.Removed specs
bar-drilldownbar-basicpie-drilldownpie-basicmap-drilldown-geographicchoropleth-basicpie-portfolio-interactivedonut-basic/donut-nestedhierarchy-toggle-viewtreemap-basic+sunburst-basicscatter-matrix-interactivescatter-matrixscatter-brush-zoomlinked-views-selectiondashboard-synchronized-crosshairline-navigatorbar-race-animatedbar-horizontalscatter-animated-controlsbubble-basicmap-animated-temporalscatter-map-geographicKept despite interactive-keyword matches (interactivity only incidental, static value intact):
candlestick-volume,contour-3d,scatter-3d,line-3d-trajectory,hexbin-map-geographic,line-annotated-events,network-force-directed,stock-event-flags,dashboard-metrics-tiles,map-tile-background. Borderlinemap-marker-clusteredkept for now.Cleanup after merge
sync_to_postgreson push to main (removes specs/impls no longer in the repo).gs://anyplot-images/plots/<spec-id>/preview images will be deleted manually after merge.🤖 Generated with Claude Code