CBICA / NiChart_Project

Neuro Imaging Chart of AI-based Imaging Biomarkers
https://neuroimagingchart.com/
Other
9 stars 3 forks source link

git clone issue due to "NiChart Project Resources" url doesn't exist #4

Closed da-ma-dm closed 3 weeks ago

da-ma-dm commented 3 weeks ago

When installing the NiChart Project using the git command git clone --recursive https://github.com/CBICA/NiChart_Project.git: It seems the submodule NiChart Project Resources page seems cannot be accessed:       https://github.com/CBICA/NiChart_Project_Resources/tree/59e3e3c1ba2cd27de4870b0118e9ce083f4b90eb This stops the installation to clone all subsequent submodules

Error message:

Failed to clone 'NiChart_Project_Resources' a second time, aborting

spirosmaggioros commented 3 weeks ago

@da-ma-dm I'm preparing a full update on the modules and organization of the repo, this will be fixed once it's done. You are correct, some submodules are not initialized properly. This submodule is private now and it will be included inside the project directly, without a submodule. Sorry for the confussion

da-ma-dm commented 3 weeks ago

Hi Spiros,

That's great to hear. Thanks a lot for your reply and feedback!

BTW, just want to mention that, when I'm trying to access: https://github.com/CBICA/NiChart_Project_Resources It shows "404" web page not found error. By any chance it might be because that the repo has not been made publically readable yet?

Thank you again!

spirosmaggioros commented 3 weeks ago

As i said, this repo is private for now due to privacy reasons. We will try to update the whole repository as soon as possible. You can remove the submodule for now git rm --force NiChart_Project_Resources, i also think the pNet submodule is not properly added as well, so you have to remove it and then git submodule add --force https://github.com/CBICA/pNet. This won't be an issue with the new update

da-ma-dm commented 3 weeks ago

That sounds great. thank you, Spiros, for the clarification!

spirosmaggioros commented 3 weeks ago

Thank you for your feedback @da-ma-dm, this is closed and will be fixed with the new update!