-
Notifications
You must be signed in to change notification settings - Fork 111
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
datalad install -r datasets -s /// all failed to operate correctly #693
Comments
actually even without recursion installation of those awkwardly reported installed (but not really installed) subdatasets resulted in infinite recursion: $> datalad install openfmri/ds000113d
2016-08-09 23:04:27,290 [INFO ] Installing /mnt/datasets/datalad/test-crawled2/all-nonrecursive/openfmri/ds000113d (install.py:375)
2016-08-09 23:04:27,481 [INFO ] Installing /mnt/datasets/datalad/test-crawled2/all-nonrecursive/openfmri/ds000113d (install.py:375)
2016-08-09 23:04:27,612 [INFO ] Installing /mnt/datasets/datalad/test-crawled2/all-nonrecursive/openfmri/ds000113d (install.py:375)
... although installs fine individually: $> datalad install ///openfmri/ds000113d
2016-08-09 23:05:31,940 [INFO ] Installing <Dataset path=/mnt/datasets/datalad/test-crawled2/ds000113d> from http://datasets.datalad.org/openfmri/ds000113d (install.py:375)
1 installed item is available at
<Dataset path=/mnt/datasets/datalad/test-crawled2/ds000113d> |
just notes to myself to prove my sanity:
but then all of a sudden, after I added a lgr.debug call started to work
commenting out debug stmt didn't help to bring that portion of the bug back .... I am really not sure WTF has happened and it seems that now original install command works... I guess it was the tired mind + stars. I will close for now |
not quite sure yet of why the issue comes about or not, but we ran into it again, and it seems at least partially (infinite recursion while installing a subdataset into not yet initialized directory -- PR to fix is coming)... |
… don't even test for it to be present This prevents infinite recursion as it was reported in datalad#693 (comment)
If you don't mind waiting a few days, I'd propose to not care for it, since I'll rework |
On Thu, 25 Aug 2016, Benjamin Poldrack wrote:
As I said, I will not stop anyone fixing install to make it better ;) |
… don't even test for it to be present This prevents infinite recursion as it was reported in datalad#693 (comment)
… don't even test for it to be present This prevents infinite recursion as it was reported in datalad#693 (comment)
Fix is deprecated with #613, but new implementation doesn't do what the fix was avoiding. So, until someone reproduces this, I'll mark this for closing. |
it did install all/openfmri (but reported installing each one twice, starting with ds000148) but for the others (crcns, etc) nothing was done and weird installation successes were reported:
and then attempt to install crcns "manually" lead to infinite recursion (until crashed)
$> datalad install -r datasets all/crcns 2016-08-09 22:43:32,235 [INFO ] Installing /mnt/datasets/datalad/test-crawled2/all/crcns (install.py:375) 2016-08-09 22:43:32,302 [INFO ] Installing /mnt/datasets/datalad/test-crawled2/all/crcns (install.py:375) ...
so there is some major difference in how later datasets got published I guess although I don't see yet obvious difference :-/
The text was updated successfully, but these errors were encountered: