Skip to content

Feature Statistics: Rename to Variable Statistics #7085

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

janezd
Copy link
Contributor

@janezd janezd commented May 20, 2025

Issue

Closes #7084.

Description of changes
  • Rename Feature Statistics to Variable Statistics.
  • Rename the column from "Name" to "Variable" in the Variable Statistics table.
  • Rename the output variable from "Feature" to "Variable".

Backward compatibility (non-)issues:

  • Renaming the output variable will break existing workflows that ... huh, that ... filtered by this variable?! Such workflows are unlikely, and the fix would introduce some ugly code, so let us not worry about it.
  • Since only the name (and not the qualified name) has changed, existing workflows won't be affected.
  • I added a keyword so that typing "feature" still finds the widget.

To do:

  • Change documentation ... after the change is confirmed and I (re-)discover how to properly clone the new repo. :)
Includes
  • Code changes
  • Documentation

@janezd janezd changed the title Feature Statitics: Rename to Variable Statistics Feature Statistics: Rename to Variable Statistics May 20, 2025
@markotoplak
Copy link
Member

Column Statistics?

@wvdvegte
Copy link

Column Statistics would also be better than Feature Statistics

@janezd janezd added the needs discussion Core developers need to discuss the issue label May 22, 2025
@janezd janezd added this to the 3.39 milestone May 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs discussion Core developers need to discuss the issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature Statistics: address inadequate naming
3 participants