Skip to content

Conversation

@jingjingwu1225
Copy link
Contributor

No description provided.

Copy link
Member

Choose a reason for hiding this comment

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

Please rename to webknossos_download.png.

Copy link
Member

Choose a reason for hiding this comment

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

Please rename to webknossos_download_options.png.

Copy link
Member

Choose a reason for hiding this comment

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

Please rename to webknossos_data_volume.png.

Copy link
Member

@kabilar kabilar left a comment

Choose a reason for hiding this comment

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

Hi @jingjingwu1225, great work here on outlining this procedure. I have some minor suggestions, but overall looks great. Thanks.

@kabilar
Copy link
Member

kabilar commented May 5, 2025

Hi @jingjingwu1225, when you have a chance, could you please review my suggestions? Thanks.

jingjingwu1225 and others added 9 commits May 5, 2025 15:28
Co-authored-by: Kabilar Gunalan <kabilar.gunalan@gmail.com>
Co-authored-by: Kabilar Gunalan <kabilar.gunalan@gmail.com>
Co-authored-by: Kabilar Gunalan <kabilar.gunalan@gmail.com>
Co-authored-by: Kabilar Gunalan <kabilar.gunalan@gmail.com>
Co-authored-by: Kabilar Gunalan <kabilar.gunalan@gmail.com>
Co-authored-by: Kabilar Gunalan <kabilar.gunalan@gmail.com>
Co-authored-by: Kabilar Gunalan <kabilar.gunalan@gmail.com>
Co-authored-by: Kabilar Gunalan <kabilar.gunalan@gmail.com>
Co-authored-by: Kabilar Gunalan <kabilar.gunalan@gmail.com>
Copy link
Member

@kabilar kabilar left a comment

Choose a reason for hiding this comment

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

Thank you @jingjingwu1225. I will go ahead and merge and submit a follow up pull request to rename the files.

@kabilar kabilar merged commit f8f071f into lincbrain:main May 9, 2025
2 checks passed
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.

2 participants