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

NKI urls for left and right hemispheres point to the same file #2117

Open
jeromedockes opened this issue Aug 9, 2019 · 6 comments

Comments

@jeromedockes
Copy link
Contributor

commented Aug 9, 2019

it seems that e.g.
https://www.nitrc.org/frs/download.php/8261/A00033747_rh_preprocessed_fsaverage5_fwhm6.gii
and
https://www.nitrc.org/frs/download.php/8261/A00033747_lh_preprocessed_fsaverage5_fwhm6.gii

point to the same file (true for other subjects as well), which corresponds to the left hemisphere.

see
https://neurostars.org/t/preprocessed-nki-fsaverage5-surface-data/4784

@juhuntenburg do you know if the files uploaded on nitrc have changed recently?

@juhuntenburg

This comment has been minimized.

Copy link
Member

commented Aug 13, 2019

Hi, just to understand the issue, what do you mean by they point to the same file (the two urls are different)?

@jeromedockes

This comment has been minimized.

Copy link
Contributor Author

commented Aug 13, 2019

the two urls are different but when I download them the result is exactly the same (try to wget both URLs then cmp the resulting files)

@juhuntenburg

This comment has been minimized.

Copy link
Member

commented Aug 14, 2019

The reason is that they have the same identifier (8261) which is not the correct one. The real files have the following urls and are different:
https://www.nitrc.org/frs/download.php/8262/A00033747_lh_preprocessed_fsaverage5_fwhm6.gii
https://www.nitrc.org/frs/download.php/8263/A00033747_rh_preprocessed_fsaverage5_fwhm6.gii

Where did you get the urls from?

@jeromedockes

This comment has been minimized.

Copy link
Contributor Author

commented Aug 14, 2019

they are the ones used by the nilearn fetcher:

archive % (nitrc_ids[i], ids[i], 'rh'),

archive % (nitrc_ids[i], ids[i], 'lh'),

@juhuntenburg

This comment has been minimized.

Copy link
Member

commented Aug 14, 2019

Jap, that is wrong, should I fix it?

@jeromedockes

This comment has been minimized.

Copy link
Contributor Author

commented Aug 14, 2019

If you can that would be great! thanks!

@juhuntenburg juhuntenburg referenced a pull request that will close this issue Aug 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.