Skip to content
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 BrainHack project "General Spinal Cord Segmentation" #253

Merged

Conversation

louisfb01
Copy link
Contributor

This is a pull request to add my BrainHack project to the gallery, following https://school.brainhackmtl.org/project_guide/.

@louisfb01 louisfb01 changed the title added project index.md and images New BrainHack project "General Spinal Cord Segmentation" Jun 5, 2023
@louisfb01
Copy link
Contributor Author

louisfb01 commented Jun 8, 2023

Tagging TA @valosekj for review :)

@valosekj
Copy link
Collaborator

valosekj commented Jun 8, 2023

Tagging TA @valosekj for review :)

Nice! I have a vacation from 9-17 June --> will review it after.

@pbellec pbellec requested review from pbellec and valosekj June 13, 2023 13:12
Copy link
Collaborator

@pbellec pbellec left a comment

Choose a reason for hiding this comment

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

I've rendered the page, and everything looks great. Congrats on the awesome project. Pending the minor comment I'll left, I'll be ready to merge. I'll wait a bit on the review by the Poly hub though.

@pbellec
Copy link
Collaborator

pbellec commented May 18, 2024

The review was never properly completed but the report looks good, so I am merging.

@pbellec pbellec merged commit ed87c65 into school-brainhack:main May 18, 2024
@valosekj
Copy link
Collaborator

Apologies for no review from my side; it somehow slipped through the cracks.

Indeed, the project looks good. Thanks for merging!

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.

None yet

3 participants