Skip to content

documentation: update headings and descriptions for api reference#1220

Merged
speller26 merged 16 commits intoamazon-braket:mainfrom
gluonhiggs:docs/headings-and-descriptions-for-api-reference
Mar 2, 2026
Merged

documentation: update headings and descriptions for api reference#1220
speller26 merged 16 commits intoamazon-braket:mainfrom
gluonhiggs:docs/headings-and-descriptions-for-api-reference

Conversation

@gluonhiggs
Copy link
Copy Markdown
Contributor

*Issue #1186

Description of changes:

  • Remove TOC heading
  • Add module descriptions
  • Add module aliases, package aliases.

About the module descriptions, I reference the aws official docs, e.g., ahs.

There are private classes in the modules that I update their descriptions. If I do not add their paths to apidoc_excluded_paths, empty pages would exist like this
Screenshot from 2026-02-16 15-23-42

Therefore, I have to explicitly list them there. If we tolerate the existence of these empty pages, we can unchange doc/conf.py.

Testing done:

Merge Checklist

General

Tests

-[x] I have run the existing tests. No breaking changes.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@gluonhiggs gluonhiggs requested a review from a team as a code owner February 16, 2026 09:40
@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 19, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (8597aa6) to head (ff36f3a).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main     #1220   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          169       169           
  Lines        10835     10835           
  Branches      1382      1382           
=========================================
  Hits         10835     10835           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@speller26
Copy link
Copy Markdown
Member

Thank you for the PR! This is looking really good! One thing: the tree should not be fully expanded from the get-go; we only need to include the top-level headings. As well, don't need the "Braket SDK" heading either; in fact, the API reference home should just be what you see when you click Braket SDK.

image

@gluonhiggs
Copy link
Copy Markdown
Contributor Author

I have pushed some changes based on your feedback. To remove the Braket SDK heading, we have to change _apidoc/modules to _apidoc/braket in index.rst. However, whenever we build html, modules.rst is always generated, and it is not used (a warning is raised). Hence, I add modules.rst to exclude_patterns in conf.py.

@speller26 speller26 self-assigned this Feb 19, 2026
Copy link
Copy Markdown
Member

@speller26 speller26 left a comment

Choose a reason for hiding this comment

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

Looking great! Just a few comments.

Comment thread src/braket/annealing/__init__.py Outdated
Comment thread src/braket/aws/__init__.py Outdated
Comment thread src/braket/circuits/__init__.py Outdated
Comment thread src/braket/program_sets/__init__.py Outdated
Comment thread src/braket/tasks/__init__.py Outdated
Comment thread src/braket/ipython_utils.py Outdated
Comment thread src/braket/experimental_capabilities/__init__.py Outdated
@gluonhiggs gluonhiggs force-pushed the docs/headings-and-descriptions-for-api-reference branch from e3d715a to 09a12b6 Compare February 28, 2026 09:11
@gluonhiggs
Copy link
Copy Markdown
Contributor Author

@speller26 I guess you have to trigger the GitHub actions workflows, right?

@speller26 speller26 merged commit 4e6629d into amazon-braket:main Mar 2, 2026
18 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.

3 participants