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

Feature Request: Not sorting the correlation matrix #1694

Closed
SemKl opened this issue Jan 20, 2025 · 3 comments
Closed

Feature Request: Not sorting the correlation matrix #1694

SemKl opened this issue Jan 20, 2025 · 3 comments
Labels
feature request 💬 Requests for new features

Comments

@SemKl
Copy link

SemKl commented Jan 20, 2025

Missing functionality

First of all, this is a wonderful tool. Thank you very much for making this.

I was wondering if there is an option, or whether it could be added, not to sort the correlation matrix alphabetically. Currently, even when calling ProfileReport(data, sort=None) (which is also the default), the order of all columns is kept in every section, except in the correlation matrix, which is still sorted alphabetically.

Proposed feature

Allow the correlation matrix to not be sorted alphabetically. I think it should be linked to the sort parameter as well.

Alternatives considered

No response

Additional context

No response

@fabclmnt fabclmnt added feature request 💬 Requests for new features and removed needs-triage labels Feb 6, 2025
@fabclmnt fabclmnt moved this to Selected for next release in YData-profiling roadmap Feb 6, 2025
@fabclmnt
Copy link
Collaborator

fabclmnt commented Feb 6, 2025

Hi @SemKl ,

thank your for your words! It is great to know the impact that ydata-profiling has!

It is certainly possible. We will consider it for the next releases.

@SemKl
Copy link
Author

SemKl commented Feb 7, 2025

Hi, thanks for your reply. I’ll look forward to it! For now, as a workaround, I prepend integers to my variable names using zfill, so it wil sort “ascending” using those numbers. Might be useful for anyone looking for a temporary solution!

@SemKl SemKl closed this as completed Feb 7, 2025
@github-project-automation github-project-automation bot moved this from Selected for next release to Approval in YData-profiling roadmap Feb 7, 2025
@SemKl
Copy link
Author

SemKl commented Feb 7, 2025

Oops, I accidentally closed this 😅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request 💬 Requests for new features
Projects
Status: Approval
Development

No branches or pull requests

3 participants