Skip to content

ENH: Allow painting all annotation vertices in a single color #273

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Aug 12, 2019
Merged

ENH: Allow painting all annotation vertices in a single color #273

merged 1 commit into from
Aug 12, 2019

Conversation

mwaskom
Copy link
Member

@mwaskom mwaskom commented Aug 12, 2019

A small enhancement that makes it easy to show the borders between annotation labels in a single color. This makes less distracting figures in cases where you want to overlay an annotation on a statistical map and either know what each parcel represents or don't precisely care.

e.g.

from surfer import Brain
b = Brain("fsaverage", "lh", "inflated")
b.add_annotation("aparc", color="dodgerblue")

annot_edges

larsoner
larsoner reviewed Aug 12, 2019
View reviewed changes
Copy link
Contributor

@larsoner larsoner left a comment

Choose a reason for hiding this comment

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

Otherwise LGTM

@larsoner
Copy link
Contributor

Agreed, thanks @mwaskom

Travis error is ContinuumIO/anaconda-issues#11195 so let's ignore it

@larsoner larsoner merged commit 80c3ccb into nipy:master Aug 12, 2019
@mwaskom mwaskom deleted the annot_color branch August 12, 2019 17:33
@mwaskom
Copy link
Member Author

mwaskom commented Aug 12, 2019

Thanks for the quick merge @larsoner

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@larsoner larsoner larsoner left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mwaskom @larsoner