neurodata / ndgrutedb

Pipeline and tools for estimating human connectomes from Diffusion, Structural, and Functional MRI
http://neurodata.io
Apache License 2.0
29 stars 13 forks source link

convert.py example requires -l flag #211

Closed j6k4m8 closed 5 years ago

j6k4m8 commented 8 years ago

The convert.py example-script won't run without the -l flag (too few arguments). Should I be using something like -l 0 or -l False?

disa-mhembere commented 8 years ago

No, this may be a bug. You should be able to post without -l. I'll try to check as i do something else right now simultaneously.

disa-mhembere commented 8 years ago

Weird .. I can see your tasks hitting the server and succeeding.

2015-12-27 00:17:45,150: INFO/MainProcess] Received task: pipeline.tasks.task_convert[2637a199-e21c-42fd-8e99-ae4651fdb127] [2015-12-27 00:17:45,154: WARNING/Worker-7] Entering convert task ... [2015-12-27 00:17:45,158: WARNING/Worker-7] Writing converted file test.edgelist ... [2015-12-27 00:17:45,159: WARNING/Worker-7] Exiting convert task ... [2015-12-27 00:17:45,174: INFO/MainProcess] Task pipeline.tasks.task_convert[2637a199-e21c-42fd-8e99-ae4651fdb127] succeeded in 0.0211818349999s: "- File: 'test.edgelist'\n- 'File conversion format... [2015-12-27 00:17:50,394: INFO/MainProcess] Task pipeline.tasks.task_mp_convert[43210f85-9d5c-4ec8-8d3b-293553f7ff7c] succeeded in 10.605077863s: None [2015-12-27 00:17:50,456: INFO/MainProcess] Task pipeline.tasks.task_mp_convert[7ed57ed9-f13c-4e56-8e57-2b6bd2885773] succeeded in 10.567229285s: None

j6k4m8 commented 8 years ago

I tried a few times with the flag as well, that may be what you're seeing.

disa-mhembere commented 8 years ago

Perhaps your input format -i is incorrect? This succeeded.

python convert.py http://openconnecto.me/graph-services/convert test.txt your@email -i graphml -o ncol,edgelist,lgl,pajek

test.txt

disa-mhembere commented 8 years ago

I should make -i andn -o positional and not optional.

j6k4m8 commented 8 years ago

Interesting. Agreed re positional args, my command is too far back in my history to pull up but it definitely had in/out first, and then URL, file, and email. i.e. not in your order above

disa-mhembere commented 8 years ago

The mail server also has a bunch of undeliverable mail to ndio-demos@neurodata.io

j6k4m8 commented 8 years ago

oh derp. yeah that's my bad. sorry about that

disa-mhembere commented 8 years ago

:) no problem, but I guess we know where all the failed jobs were going (or actually not going). Most of them were unknown format though:

[...] Here's an error list:

On Sun, Dec 27, 2015 at 12:42 PM, Jordan Matelsky notifications@github.com wrote:

oh derp. yeah that's my bad. sorry about that

— Reply to this email directly or view it on GitHub https://github.com/openconnectome/m2g/issues/211#issuecomment-167430426.