Skip to content

Fix Sphinx currentmodule directive for location choice documentation#1034

Merged
jpn-- merged 2 commits intomainfrom
copilot/fix-missing-doc-content
Feb 12, 2026
Merged

Fix Sphinx currentmodule directive for location choice documentation#1034
jpn-- merged 2 commits intomainfrom
copilot/fix-missing-doc-content

Conversation

Copy link
Contributor

Copilot AI commented Feb 12, 2026

The Implementation sections for work_location_choice and school_location_choice were not rendering because the currentmodule directive pointed to non-existent submodules.

Changes

  • work_location_choice.md: Corrected currentmodule from activitysim.abm.models.location_choice.workplace_location to activitysim.abm.models.location_choice
  • school_location_choice.md: Corrected currentmodule from activitysim.abm.models.location_choice.school_location to activitysim.abm.models.location_choice

The functions workplace_location and school_location are functions within the location_choice module, not submodules themselves. The fix aligns these docs with the pattern used throughout other component documentation (e.g., auto_ownership, mandatory_tour_frequency).

Original prompt

This section details on the original issue you should resolve

<issue_title>missing doc content</issue_title>
<issue_description>Describe the bug
The "implementation" section is not rendering any content.

To Reproduce
https://activitysim.github.io/activitysim/develop/dev-guide/components/work_location_choice.html#implementation

Expected behavior
Similar to other components, the API of this model should appear there.

As reported by @guyrousseau via email</issue_description>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

…ation_choice.md

Co-authored-by: jpn-- <1036626+jpn--@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix missing content in implementation section Fix Sphinx currentmodule directive for location choice documentation Feb 12, 2026
Copilot AI requested a review from jpn-- February 12, 2026 21:30
@jpn-- jpn-- marked this pull request as ready for review February 12, 2026 22:30
@jpn-- jpn-- merged commit 83613b2 into main Feb 12, 2026
2 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.

missing doc content

2 participants