Add audio-analysis coverage panel to provider settings#1783
Open
chrisuthe wants to merge 14 commits into
Open
Conversation
…anel Fetch once on mount; users reload the page for updated stats. Removes startAutoRefresh/stopAutoRefresh from the composable, the refresh button and poll lifecycle from the component, the now-dead refresh i18n key and CSS, and the polling test.
Contributor
|
Screenshot? |
…e state Adopt the audio_analysis/status endpoint as the authoritative loaded-state and analysis_version probe per provider, then fetch audio_analysis/coverage only for loaded providers. A rejecting or provider_loaded:false status now overrides an optimistic api.providers flag (row marked unavailable, no coverage call); a loaded provider whose coverage call fails still surfaces as an error row with the version from status.
Replace the hand-rolled table and scoped hex CSS with Card/Item/Progress/Badge primitives and semantic Tailwind tokens, matching the settings visual standards (cf. About.vue). No data or composable changes; test DOM contract and i18n keys preserved.
… of the page The coverage panel was mounted raw at the top of Providers.vue with no wrapper, so it bled to the viewport edges while the header/total/list below all use the page's 20px (px-5) gutter. Wrap the mount in px-5 pt-5 to match; component stays layout-agnostic.
Contributor
|
Nice. Not sure how easy it would be to add my AcoustID provider. We would have to know how many tracks are missing a musicbrainz recording ID first…. |
Contributor
|
I think it is a bit weird to show this by default at the top of the screen. Let's make a button next to 'add provider' that just opens a new breadcrumb settings > audio analysis > coverage (or something) that shows this overview? |
chrisuthe
added a commit
to chrisuthe/frontend
that referenced
this pull request
May 20, 2026
Coordinate with frontend PR music-assistant#1783 (audio-analysis coverage panel) and server PR #3851 (centralised audio_analysis API): - Remove per-row AA status badges from Providers.vue + the settings.aa_provider.* strings; PR music-assistant#1783's coverage panel owns this surface and would render duplicate information. - Rewire getSonicAnalysisStatus() to audio_analysis/status with aa_domain=sonic_analysis, and add getAaProviderCoverage(aa_domain) for audio_analysis/coverage. Drop the unused getAaProviderStatus() helper that only existed to feed the per-row badges. - Refactor SonicSimilarityStatusPanel: read aa_provider_domain from sonic_similarity/status, then fetch audio_analysis/coverage for that domain. Coverage % is now index_size / (analyzed + pending) and works for any AA backend (sonic_analysis today, loudness later) without code changes. Add the AaProviderCoverage interface ({analyzed, pending, stale_version, analysis_version}) matching #3851's response shape.
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
3 participants
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.
Adds a self-contained coverage panel above Search/add-provider on /settings/providers?types=audio_analysis: per-provider analyzed/pending coverage bar, stale & version columns, plus a read-only background-scan status line.
Sourced entirely from existing APIs (audio_analysis/coverage, tasks/get) — no server changes.
Depends on music-assistant/server#3851