-
Notifications
You must be signed in to change notification settings - Fork 13
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.
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
New set of checkboxes for visible features #59
Comments
Thanks Pradeep! Here's suggestion of check boxes with a couple of very minor modifications to your proposal
I think "too much Brow" is focused for our/DSST needs but rest are good prompts for general defacing QC. Also, I removed "facial features" since we are specifying eyes, ears and brow, which covers the more common points of under-stripping. I can help with this but I was wondering if we could document and provide a couple of examples for each category that's not pass or review later, so that users know what we mean by "too much ears" etc? |
Good suggestions Arsh, I'll add the Brow visible. More words in option might make the checkbox too long wasting space. if you could supply some examples to documentation, that would be great. Do you know how to work with |
It's been a while since I used ReST but it shouldn't be a problem to figure it out. Thanks, Pradeep. I'll try to get it done by next week! |
Great, thanks Arsh. Also, depending on your interest/skill in Python, it would be straightforward to integrate freeview in VisualQC, given it works on your system now. If you are interested in contributing, i can walk you through what needs to be done (simply a loop calling fsleyes and managing its behaviour correctly). It shouldn't take more than a day or two. |
Sure, that sounds good but did you mean fsleyes or freeview? I haven't used freeview before but it shouldn't be too hard to figure it out. |
LOL, my bad, I meant fsleyes :) |
we will be imitating how the freesurfer module generated cortical visualization using freeview or tksurfer, but now using fsleyes -- rather straightforward to change command name and parameters: visualqc/visualqc/freesurfer.py Line 342 in 46ff96f
|
some facial features may still be visible after defacing such as eyes and ears, it would be help capture additional info
The text was updated successfully, but these errors were encountered: