Skip to content

[ENH] Change some visual bundle colors for pyAFQ#192

Merged
36000 merged 5 commits into
mainfrom
nongraypaf
May 25, 2026
Merged

[ENH] Change some visual bundle colors for pyAFQ#192
36000 merged 5 commits into
mainfrom
nongraypaf

Conversation

@36000

@36000 36000 commented May 21, 2026

Copy link
Copy Markdown
Collaborator

Change pAF color to something non-gray, so that it can be better overlayed on gray backgrounds
Specify OR and TP colors. All of these are more neon than our typical colors.

Copilot AI review requested due to automatic review settings May 21, 2026 21:09
@36000

36000 commented May 21, 2026

Copy link
Copy Markdown
Collaborator Author

@copilot update fibertracts.rst in docs to correspond with these changes

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

Updates pyAFQ’s visualization color palette to make specific bundles more visible (particularly against gray backgrounds) and to explicitly define colors for Optic Radiations and “TP”.

Changes:

  • Updated Left/Right Posterior Arcuate colors from gray shades to more vivid RGB values.
  • Added explicit RGB colors for Left/Right Optic Radiation.
  • Added new RGB entries for Left/Right Temporo-parietal.

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

Comment thread AFQ/viz/utils.py
Comment thread AFQ/viz/utils.py

Copilot AI commented May 21, 2026

Copy link
Copy Markdown
Contributor

@copilot update fibertracts.rst in docs to correspond with these changes

Addressed in 4155e69. I updated docs/source/reference/fibertracts.rst to reflect the new neon colors for Left/Right Superior Longitudinal and Left/Right Optic Radiation, and added Left/Right Temporo-parietal entries. I also added matching color classes in docs/source/_static/custom.css so the roles render correctly.

Copilot finished work on behalf of 36000 May 21, 2026 21:17
@36000

36000 commented May 21, 2026

Copy link
Copy Markdown
Collaborator Author

@copilot great job, could you also update posterior arcuate?

Copilot AI and others added 2 commits May 21, 2026 21:49

Copilot AI commented May 21, 2026

Copy link
Copy Markdown
Contributor

@copilot great job, could you also update posterior arcuate?

Done in cbb5826 (with cleanup in e7464b4). Posterior Arcuate is now updated in docs/source/reference/fibertracts.rst to match the new colors, and corresponding CSS classes were added in docs/source/_static/custom.css.

Screenshot: https://github.com/tractometry/pyAFQ/blob/nongraypaf/docs/source/reference/fibertracts.rst#L201-L208

Copilot finished work on behalf of 36000 May 21, 2026 21:50
@36000
36000 merged commit 7d946e9 into main May 25, 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