Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

docs: add study entity PRD (#4376)#4701

Open
NoopDog wants to merge 1 commit into
main from
noopdog/4376-study-entity-prd
Open

docs: add study entity PRD (#4376) #4701
NoopDog wants to merge 1 commit into
main from
noopdog/4376-study-entity-prd

Conversation

@NoopDog

@NoopDog NoopDog commented Mar 14, 2026
edited
Loading

Copy link
Copy Markdown
Collaborator

Summary

  • Add PRD for the new /index/studies entity endpoint in Azul, specifying new hit fields (from DUOS), roll-up aggregation rules from child entities, and termFacet propagation
  • Define how study-level data flows to child entity endpoints following existing Azul patterns (summary hit objects + full facet propagation)
  • Include example API responses and coverage analysis scripts verified against live DUOS/Azul data

Test plan

  • Review PRD for completeness against existing Azul entity patterns
  • Verify DUOS field paths using the included coverage scripts
  • Confirm example JSON responses match expected API shape

🤖 Generated with Claude Code

@NoopDog NoopDog force-pushed the noopdog/4376-study-entity-prd branch 5 times, most recently from 3077688 to 3c84b00 Compare March 14, 2026 16:17
@NoopDog NoopDog requested a review from Copilot March 15, 2026 23:07

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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

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 5 comments.


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

Comment thread prds/study-entity/scripts/study_coverage.py
Comment thread prds/study-entity/example-studies-response.json
Comment thread prds/study-entity/example-datasets-response.json
Comment thread prds/study-entity/scripts/consortia_coverage.py
Comment thread prds/study-entity/scripts/study_coverage.py
Add PRD for the new /index/studies entity endpoint in Azul, specifying
new hit fields from DUOS, roll-up aggregation rules from child entities,
and termFacet propagation to all entity endpoints.
Includes example API responses verified against live DUOS/Azul data
and Python scripts for coverage analysis.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@nadove-ucsc nadove-ucsc Awaiting requested review from nadove-ucsc

@hannes-ucsc hannes-ucsc Awaiting requested review from hannes-ucsc

At least 1 approving review is required to merge this pull request.

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /