Repository navigation
App: replace Agent Evaluation and Leaderboards with one Agents page - #184
Open
Keith McGrane (Keithland89) wants to merge 1 commit into
Open
Keith McGrane (Keithland89) wants to merge 1 commit into
Keith McGrane (Keithland89) wants to merge 1 commit into
Conversation
Analytics Hub keeps its current design. The Agent Evaluation and Leaderboards pages, and their queries, are removed. One Agents page replaces them: agents in use against registered, people using agents weekly, people by agent type, and a searchable, exportable table of the top 200 agents, sorted by people and then sessions. Each row opens the agent's details, with an "Open in Agent Evaluator" link to the AE report when the install names one, or to its model otherwise. The top 1%, 5% and 25% session concentration moves to Adoption. Ask, the Azure Ask catalogue, the installer labels and the docs follow the change. The installer's opt-in Agent Evaluator step is unchanged. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
3 of 5 tasks
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
The Analytics Hub app keeps its current design. The Agent Evaluation and Leaderboards pages are removed, and one Agents page replaces them, as the design review recommended.
The Agents page (Value Lens model, with every existing filter):
agentLeaderboard), sorted by People, then Sessions;reportIdon theaeentry in the runtime config, kept only when it's a GUID), and otherwise the model.reportIdto the app. Both PRs should merge in the same release.Other changes:
leaderboard-concentration) moves here, under the weekly trend, with a one-line headline.azure/web/src/ask.js), the installer's page labels and the data agent's text: these now refer to the Agents page.aemodel; it only links to it.Not taken from the review, as Keith asked: no filter banner, no page merges, and no palette or Assumptions changes.
Scope
Parts affected:
app/src.steps/app.jsandtransform/data-agent.js.There are no Fabric, Azure infrastructure, job, core or Power BI model changes.
Source contracts, optional inputs, install record or deployed item names changed:
agent-evaluation/*,evaluator-freshnessand the leaderboard page queries.agents/agent-weekly,agent-types,agent-trend,agent-orgsandagent-detail. They use the model's existing measures at the grain each view needs, with no raw-row pulls.agentUsage, for Ask;agentLeaderboardandleaderboard-cohortsquery names;semanticModels.<alias>may now carry an optionalreportId. The app ignores it unless it's a GUID. Nothing writes it yet.aealias and the deployed item names are unchanged.Validation
python -B -m pytest tests -qand recorded results below.npm test(andnpm run typecheckfor the installer) in each changed Node project..pbitfiles changed, documented the rebuild method and Desktop refresh / visual checks. (No.pbitchanges.)Results, skipped checks and limitations:
npx tsc -b: passed.npm test: 3,432 of 3,433 passed. One test,report-export.spec.ts > deck > builds a PowerPoint file, timed out at 6.5 s while the installer suite ran in parallel. Re-run alone,npx vitest run src/lib/report-export.spec.tspassed 16/16. A separate earlier full run passed 3,433/3,433.npm run lint: 0 errors and 2 warnings, the same as main.npm run build:fabric: passed.npm run typecheck: passed.node --test: 1,075 of 1,076 passed. The one failure,script: generated Windows commands run…, happens because this machine's PowerShell execution policy blocks the generated script. It's environmental, and I didn't bypass the policy.node --test "test/**/*.test.js": 84 of 84 passed.python -B -m pytest tests/test_docs.py tests/test_executive_query_measures.py tests/test_parked_ideas.py tests/test_dax_var_names.py tests/test_dax_sort_by_columns.py tests/test_consolidated_release.py tests/test_agent_evaluator.py tests/test_azure_agent_evaluator.py -q: 294 passed. These are the files that read the app queries, docs and Agent Evaluator. The branch changes no Python. The fullpython -B -m pytest tests -qrun: 1258 passed, 5 skipped (57 min).[Active Agent Users],[% Active Users Using Agents],[Observed Agent Sessions],[Observed Agent Sessions per User],[Observed Agent Sessions per User per Week],[Agent Registry Records]and the existing leaderboard and concentration queries). The query specs pin each query's DAX and column mapping, and the fixtures cover the row shapes. Concentration percentages were checked by hand against the harness inputs (1,840 / 9,600 = 19%; 3,910 / 9,600 = 41%; 7,020 / 9,600 = 73%). I haven't run this against a live model; doing so means deploying, which needs Keith's OK./groups/<ws>/reports/<id>when areportIdis set.Compatibility
Repair or change redeploys the app with the new pages, and nothing else changes:
aealias and its Agent Evaluator model stay as they are.Migration:
Deliberately unchanged:
transcripts.py;