Skip to content

feat: add access metrics section to HCA analytics report#4880

Merged
NoopDog merged 2 commits into
mainfrom
mim/4879-hca-access-metrics
Jun 29, 2026
Merged

feat: add access metrics section to HCA analytics report#4880
NoopDog merged 2 commits into
mainfrom
mim/4879-hca-access-metrics

Conversation

@MillenniumFalconMechanic

Copy link
Copy Markdown
Contributor

Ticket

Closes #4879

Summary

  • Add DUOS access request tracking to HCA analytics report via access_request_urls=["duos.org"]
  • Normalize page paths to entity base path so counts are grouped by project (e.g., /projects/UUID/get-curl-command and /projects/UUID/project-metadata merge into /projects/UUID)
  • Conditionally hide summary cards and Service column when only one access request service exists (HCA has DUOS only; Anvil keeps both cards and column since it has DUOS + dbGaP)
  • Dynamic entity label in table header (shows "Project" for HCA, "Dataset" for Anvil)

Test plan

  • Regenerated HCA analytics site and verified access requests table renders with 3 grouped project rows
  • Verified no summary cards appear (single service)
  • Verified Service column is hidden (single service)
  • Verified table header shows "Project" not "Dataset"

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds DUOS-based access request tracking to the HCA analytics static site, including normalized project page-path grouping and UI adjustments to hide service-related summary/columns when only one access-request service exists.

Changes:

  • Enable access request tracking for HCA by configuring access_request_urls=["duos.org"] and generating access_requests.json.
  • Normalize access-request page_path values to the entity base path (e.g., /projects/<uuid>), improving grouping/aggregation.
  • Update the static-site template/UI to conditionally hide the access-request service summary cards/column when only one service is present, and set a dynamic entity label in the access-requests table header.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
gh-pages/hca-explorer/index.html Updates generated static site JS to support service display logic and dynamic entity labeling in the access-requests table.
gh-pages/hca-explorer/data/meta.json Updates generated timestamp metadata for the HCA analytics site build.
gh-pages/hca-explorer/data/access_requests.json Adds generated DUOS access request metrics grouped by project page path.
analytics/hca-explorer-sheets/generate_static_site.py Enables access request tracking for HCA via access_request_urls=["duos.org"].
analytics/analytics_package/analytics/static_site/template/index.html Updates template logic for access-request services/cards/columns and dynamic entity labeling.
analytics/analytics_package/analytics/static_site/fetch.py Normalizes access-request page paths to entity base paths before aggregation.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread analytics/analytics_package/analytics/static_site/fetch.py
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 4 comments.

Comment thread gh-pages/hca-explorer/index.html
Comment thread gh-pages/hca-explorer/index.html
@NoopDog NoopDog merged commit 15813b6 into main Jun 29, 2026
4 checks passed
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.

Add access metrics section to HCA analytics report

3 participants