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

DOC: Adds Iñigo's final report for GSoC2024 (Final Work Product) #67

Merged
merged 4 commits into from
Aug 25, 2024

Conversation

itellaetxe
Copy link
Contributor

@itellaetxe itellaetxe commented Aug 22, 2024

Adds my last report for GSoC2024.

Copied Kaustav's override.css modifications and images from his PR #66

Copy link
Contributor

github-actions bot commented Aug 22, 2024

🪓 PR closed, deleted preview at https://github.com/dipy/preview-html/tree/main/dipy.org/pull/67/

@itellaetxe
Copy link
Contributor Author

itellaetxe commented Aug 22, 2024

The preview seems to be okay but when I go to the blogs, the report does not appear. Suggestions please? @deka27 @skoudoro

EDIT: SOLVED. Thanks @deka27, you are an HTML wizard.

Copy link
Member

@robinroy03 robinroy03 left a comment

Choose a reason for hiding this comment

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

Good work Iñigo. Your work is inspiring. LGTM. Approving.


I am super grateful for the opportunity to participate in the Google Summer of Code program with DIPY. This journey has been amazing as learning and research experience. I am proud of the progress I have made and what I have achieved. Nevertheless, this is just the beginning of my journey with this project.

I would like to thank again my mentors, Jon Haitz, Jong Sung and Serge, their time, patience, attention, support, and expertise throughout the program. They have been a key part of my success. On the other hand, I would like to thank my fellow GSoC students, Kaustav, Wachiou, and Robin, for their support and collaboration. It has been a pleasure to work with them and to learn from them.
Copy link
Member

Choose a reason for hiding this comment

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

It was awesome to learn from you too!

Copy link
Member

Choose a reason for hiding this comment

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

Thank you @itellaetxe from everything

Copy link
Member

Choose a reason for hiding this comment

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

@itellaetxe It would be cool if you could quote them with their Github profiles. 😇

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thank you @robinroy03, @WassCodeur!
I will link everyone's names to their GitHub profiles, okay. Thanks

posts/2024/2024_08_22_inigo_final_report.rst Show resolved Hide resolved
posts/2024/2024_08_22_inigo_final_report.rst Outdated Show resolved Hide resolved
posts/2024/2024_08_22_inigo_final_report.rst Outdated Show resolved Hide resolved
posts/2024/2024_08_22_inigo_final_report.rst Outdated Show resolved Hide resolved
posts/2024/2024_08_22_inigo_final_report.rst Outdated Show resolved Hide resolved
posts/2024/2024_08_22_inigo_final_report.rst Outdated Show resolved Hide resolved
Copy link
Member

@WassCodeur WassCodeur left a comment

Choose a reason for hiding this comment

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

Well done

@pjsjongsung
Copy link

Looks great! One thing, there seems to be some markdown equation issues not resolved.

:align: center
:width: 600

The bottom row shows two sets of unseen plausible streamlines, run through the model (encode & decode). We can see that the reconstruction fidelity is not as good as the vanilla AE, but it is still acceptable, considering that the model was only trained for 120 epochs, which took around 2 hours in my GPU-less laptop.

Choose a reason for hiding this comment

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

I think this needs to change a bit because the figure changed

Copy link
Contributor Author

Choose a reason for hiding this comment

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

true, thanks

@skoudoro
Copy link
Member

Also, you need to squash your commits and rebase since @deka27 PR has been merged and create some conflict with yours.

Thanks !

Copied Kaustav's override.css modifications and images
@itellaetxe
Copy link
Contributor Author

itellaetxe commented Aug 25, 2024

Also, you need to squash your commits and rebase since @deka27 PR has been merged and create some conflict with yours.

Thanks !

@skoudoro All done. Ready to merge!

@skoudoro skoudoro merged commit fd7c2d2 into dipy:master Aug 25, 2024
3 checks passed
@skoudoro
Copy link
Member

All good 👍 ! Thank you for this work !

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.

6 participants