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

[features-] add command to hide repetitive columns #2453

Merged
merged 2 commits into from
Oct 7, 2024

Conversation

midichef
Copy link
Contributor

@midichef midichef commented Jul 13, 2024

This is a command to hide any column that contains one value repeated in every row. Is this worth having? It's often the first command I run when opening large tables with such uninformative columns.

I'm not sure what to call the command. Right now it is hide-unvarying-cols, but perhaps hide-uniform-cols is better? hide-repetitive-cols is too hard to spell.

@frosencrantz
Copy link
Contributor

This would be a useful command. Thank you for proposing it @midichef!

I like it and would use it. Saul provided a useful suggestion when I asked in Issue 431 (2022 issues ago):

[howto] hide invariant columns

visidata/features/dedupe.py Outdated Show resolved Hide resolved
@midichef midichef force-pushed the hide_boring_cols branch 2 times, most recently from 2613113 to ec87287 Compare September 29, 2024 10:29
@midichef
Copy link
Contributor Author

I changed the command prefix to hide-uniform- instead f hide-unvarying-. One reason is, "unvarying" is hardly ever used as a technical term, so it would be a new word for users to to recall. Another is, "hide" and "un-varying" describe the result based on two negatives (the positive equivalents would be "show" and "varying"). "uniform" seems better as it describes the targets of "hide" in terms of what they are (uniform), instead of what they aren't (varying).

If there's a common word that means "having one distinct value", I'd like to know.

@midichef midichef marked this pull request as ready for review September 29, 2024 10:41
@saulpw
Copy link
Owner

saulpw commented Sep 29, 2024

I changed the command prefix to hide-uniform- instead f hide-unvarying-.

I was considering suggesting 'invariant' but I think I like uniform better (less formal). Thanks for giving it some thought.

@anjakefala anjakefala merged commit c26d51d into saulpw:develop Oct 7, 2024
13 checks passed
@midichef midichef deleted the hide_boring_cols branch October 9, 2024 21:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants