Use limit=false by default in show?

[nalimilan]

The show method for Array defaults to :limit => false when the I/O stream/context doesn't set the property (notably stdout). Maybe we should do the same for consistency. See discussion at #1627.

[bkamins]

@ronisbr - is this still an issue with PrettyTables.jl?

[ronisbr]

Yes, the behavior is the same as before. If I use show without an io, then the output gets cropped by default.

[bkamins]

Then do you think it is worth to keep it open? (or maybe if it makes sense in general - maybe move the issue to PrettyTables.jl?)

[nalimilan]

One advantage of having show(df) not crop output is that people would easily have access to two different output formats: df at the REPL gives the cropped output, and show(df) would give the full output (instead of having to use e.g. show(df, allrows=true). But the output isn't readable when columns don't fit in the screen, so that wouldn't be very useful. Maybe we could use allrows=true but allcols=false by default though.

TBH I'm not sure why arrays behave like they do. The result is generally unreadable even for small ones.

[bkamins]

I would just pick what we believe is useful. Being super consistent with Base here is not a top priority I think.

[nalimilan]

But what is useful? :-) Maybe make a poll in #data?

[bkamins]

sure why not. Could you do it please? (as I believe you know exactly which case is important to compare)

[ronisbr]

Then do you think it is worth to keep it open? (or maybe if it makes sense in general - maybe move the issue to PrettyTables.jl?)

This cannot be fixed inside PrettyTables.jl. It is related to the default configurations of allrows and allcols depending on how show is called.

[bkamins]

OK - so let us keep it open here.