-
Notifications
You must be signed in to change notification settings - Fork 3
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
Hard-to-read structured output from "datalad ls" #18
Comments
@TobiasKadelka I have transferred this issue to It would indeed be good, if ATM |
@mih:
If `addurls` would additionally also support eating a JSON stream instead of an array of objects, they might even get piped together.
I've had a local to-do on the back burner regarding this. I'll promote it to a DataLad issue since it came up here. (In the case of listing a s3 bucket, though, I'd be a bit leery of feeding the output to addurls without inspection.)
|
IMHO it also relates to datalad/datalad#2126 since that one should feed records into pyout so it would become just a matter of switching "output formatter" |
Regarding original issue - let me know in which language you would prefer to see the one or two liner to parse it? Having said that may be indeed easiest just to write targeted code using boto in Python so it goes faster without trying to validate URLs etc |
IMHO it also relates to datalad/datalad#2126 since that one should feed records into pyout so it would become just a matter of switching "output formatter"
I don't think pyout is a viable solution to this issue. It (1) has a blocker [0] that I don't have any good idea on how to unblock and (2) doesn't currently have the ability to output json records (and that's not trivial to do with the current implementation).
[0]: ReproNim/reproman#400 (comment)
|
Regarding original issue - let me know in which language you would prefer to see the one or two liner to parse it? Having said that may be indeed easiest just to write targeted code using boto in Python so it goes faster without trying to validate URLs etc
FWIW the second option sounds like the better one to me.
|
Re pyout - sorry I want clear. I didn't want pyout to output json , but changes in that pr could be relevant as when to restructure code so we could could switch output renderer where one could be pyout consuming the records, and another one just print them out and json |
Re pyout - sorry I want clear. I didn't want pyout to output json , but changes in that pr could be relevant as when to restructure code [...]
Oops, that was pretty clear reading it again. Fair enough. Sorry for the noise.
|
@mih When using
the output is hard to read and parse, which makes it hard to use this information.
The text was updated successfully, but these errors were encountered: