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

glimpse vs str on factors #1454

Closed
hrbrmstr opened this issue Oct 15, 2015 · 6 comments
Closed

glimpse vs str on factors #1454

hrbrmstr opened this issue Oct 15, 2015 · 6 comments

Comments

@hrbrmstr
Copy link

currently, glimpse just gives (fctr) (keeping to the "4 or less" rule) for factor columns but with str you at least have an idea of the # of levels for the factor. is it worth violating the "4 or less" rule for factors by doing something like (fctr:3) or (fctr:10) so there is slightly more info? It's an easy tweak and I could PR that + #1325 if no one is currently working on glimpse.default.

@hadley
Copy link
Member

hadley commented Oct 15, 2015

Hmmmm - I'd prefer to keep it short so it can also go in the row header etc.

@hrbrmstr
Copy link
Author

possibly, then, controlled via an option with the default being the current short standard? or make it a glimpse-only behavior (I realize that wld be a tiny bit more plumbing). No worries either way, it just came up in a few conversations over the past few weeks and most recently today on SO.

@hadley
Copy link
Member

hadley commented Oct 15, 2015

I really hate options. Another possibility would be to come up with a shorter form: maybe <fac:3> or even <fc:3>, or even <f:3>? I suspect those are all getting a bit obscure though.

Yet another possibility would be to have another function for secondary info: could be number of factors, TZ for date times etc.

@hrbrmstr
Copy link
Author

I like the second one. Perhaps a peruse (implying more than a glimpse) that provides just slightly more info for each type?

@hadley
Copy link
Member

hadley commented Oct 15, 2015

It would need to be parallel to type_sum not glimpse, so maybe type_attr_sum ?

@hadley
Copy link
Member

hadley commented Oct 21, 2015

I'd be happy to review a PR for this.

@hadley hadley closed this as completed Oct 21, 2015
@lock lock bot locked as resolved and limited conversation to collaborators Jun 9, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants