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

Null values support #215

Merged
merged 19 commits into from
Aug 23, 2024
Merged

Null values support #215

merged 19 commits into from
Aug 23, 2024

Conversation

Tomansion
Copy link
Member

@Tomansion Tomansion commented Aug 19, 2024

Describe your changes

  • The columns can now have missing values inside them
  • Checked that all widgets support columns with missing values
    • Parallel coordinates null values line is missing (cf images)
    • Distribution plot shows “null” for classes, ignores values for numbers
    • Point plot scatter don't displays points with missing data
    • Statistics ignore missing values and display % of missing values
    • Range slider hides missing points, same for the background
    • Interval plot works, but the shape is broken (won't fix)
    • Correlation matrix plot null values are considered as 0 values
    • Confusion matrix plot null values are considered as 0 values
    • Density plot ignores null values
    • Night stars null values are considered as 0 values
    • Sample Array displays the null values properly
  • We can now filter values on null values
  • Added percentage of missing values in columns

Screenshots (if appropriate):

image
image
image
image
image
image
image

Issue ticket number and link

Checklist before requesting a review

  • I have performed a self-review of my code
  • I have increased the version number in the frontend/package.json and backend/swagger.yaml files
  • I have checked that Black, Flake8, Prettier and Cspell are passing

@Tomansion Tomansion linked an issue Aug 19, 2024 that may be closed by this pull request
5 tasks
@Tomansion Tomansion marked this pull request as ready for review August 23, 2024 08:51
@Tomansion Tomansion merged commit 13311c9 into main Aug 23, 2024
10 checks passed
@Tomansion Tomansion deleted the 191-null-values-support branch August 23, 2024 08:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Null values support
1 participant