-
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
integrate fsleyes to generate rendered images for defacing module #61
Comments
Hi @Arshitha, great meeting you in person at OHBM :). did you get a chance to work on this yet? Happy to do another zoom chat or code-review your progress so far. Let me know. |
Hi @raamana - same, it was really nice to meet you in-person! I worked on it a bit during the conference but nothing substantial yet. I've carved out some time this Friday to pick up where I left off. I'll take up on your offer for a zoom call if I get stuck. |
great, looking fwd to see this done :) |
@raamana Hello again! I spent some time on the codebase today and added the 3D renders function and modified the CLI. I have not tested it yet but I'm fairly certain it won't work as it stands. Can we schedule a meeting for next meet so I can show you what i've done so far and also, ask questions? I can send you an email to setup a meeting. If you'd like to check out the changes I've made, they're here https://github.com/Arshitha/visualqc/tree/cli-and-3d-renders |
great to hear the progress, Arshitha! happy to meet later this week, but I'd suggest you give it a try to test, and see what errors you run into (beyond the basics of syntax and stringing the right functions together). That way, we can fix the issues. Its fine if you can't string them yet for lack of clarity or other issues, just let me know what they are. I'm available in the evenings of Tuesday or thursday. |
I'd start with testing the if you already tested FYI: A friend also sent us another way to generate renders btw, and we can keep that as another option in the future, depending on how easy or hard fsleyes proves to be across platforms and software versions etc |
I've tested |
Sounds good Arshitha :) |
thanks to Arshitha and Eric from NIH
The text was updated successfully, but these errors were encountered: