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

No more parsable output for Dict printing #7142

Closed
kmsquire opened this issue Jun 5, 2014 · 4 comments · Fixed by #7147
Closed

No more parsable output for Dict printing #7142

kmsquire opened this issue Jun 5, 2014 · 4 comments · Fixed by #7147

Comments

@kmsquire
Copy link
Member

kmsquire commented Jun 5, 2014

So, while I like the new format introduced by #5706, it does seem that there isn't a way to get a compact version of the dictionary that can be parsed anymore. See

JuliaIO/JSON.jl#72

cc: @paulanalyst, @mbauman, @JeffBezanson

@mbauman
Copy link
Sponsor Member

mbauman commented Jun 5, 2014

Ah, sorry about this. Some of the old functionality is still there in showcompact. Or, well, it should have been. In a final pass to clean up the code style, I messed up the comma printing - see #7144. 💢

But it's still not quite right as it prints the summary information within the show method. Arrays were recently changed to do this within the REPL's writemime; I can make that happen here, too.

@JeffBezanson
Copy link
Sponsor Member

Good point; the new format should only be used by display.

@mbauman
Copy link
Sponsor Member

mbauman commented Jun 5, 2014

I don't know that there's much purpose to having a showcompact definition; perhaps I should just remove the summary information from it and change it to be print instead?

Or would you like a bigger change here?

@paulanalyst
Copy link

But howJSON.parse (o) savein an array ? Row by row ...?

Paul ?

W dniu 2014-06-05 21:28, Matt Bauman pisze:

I don't know that there's much purpose to having a |showcompact|
definition; perhaps I should just remove the summary information from
it and change it to be |print| instead?

Or would you like a bigger change here?


Reply to this email directly or view it on GitHub
#7142 (comment).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants