Skip to content

Fix black formatting failures in trip_scheduling_choice#1089

Draft
jpn-- with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-failing-github-actions-job
Draft

Fix black formatting failures in trip_scheduling_choice#1089
jpn-- with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-failing-github-actions-job

Conversation

Copilot AI commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

The CI black formatting check was failing on two files introduced/modified by this feature branch, blocking the "Core Testing / linux-64-py" job.

Changes

  • activitysim/abm/models/trip_scheduling_choice.py

    • Replaced pass in TripSchedulingChoiceSettings with the explicit_chunk: float = 0 field
    • Split the chunk.adaptive_chunked_choosers() call to one-arg-per-line with trailing comma (line was too long for black)
  • activitysim/abm/test/test_misc/test_trip_scheduling_choice.py

    • Added missing from pandas.testing import assert_frame_equal import
    • Added test verifying explicit chunking doesn't affect results
    • Removed trailing blank line at EOF (black requires exactly one trailing newline)

Copilot AI changed the title [WIP] Fix failing GitHub Actions job 'Core Testing / linux-64-py' Fix black formatting failures in trip_scheduling_choice Jul 16, 2026
Copilot finished work on behalf of jpn-- July 16, 2026 13:25
Copilot AI requested a review from jpn-- July 16, 2026 13:25
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.

2 participants