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

feat: univariate plotting and outlier removal #13

Merged
merged 5 commits into from
Apr 15, 2024

Conversation

ritikgupta65
Copy link
Contributor

@ritikgupta65 ritikgupta65 commented Apr 14, 2024

Description

I plotted histogram, KDE plot, and boxplot to detect outliers. Afterward, I capped the outliers that were identified.

Semver Changes

  • Patch (bug fix, no new features)
  • Minor (new features, no breaking changes)
  • Major (breaking changes)

List any issues that this pull request closes.

#3

Checklist

Copy link
Collaborator

@TheMimikyu TheMimikyu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Kindly use Histogram for discrete valued features instead of KDE Plot.

@TheMimikyu TheMimikyu changed the title Outlire ritik feat: Univariate plotting and Outlier removal Apr 15, 2024
Copy link
Collaborator

@TheMimikyu TheMimikyu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Thanks for your contribution.

@TheMimikyu TheMimikyu added PR:Accept PR has been accepted and is merged Semver:minor PR will force a minor update in repo and includes new features and backward compatibility Type:Medium PR is accepted with difficulty level as medium labels Apr 15, 2024
@TheMimikyu TheMimikyu merged commit 3b63252 into OPCODE-Open-Spring-Fest:main Apr 15, 2024
4 of 8 checks passed
@TheMimikyu TheMimikyu changed the title feat: Univariate plotting and Outlier removal feat: univariate plotting and outlier removal Apr 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR:Accept PR has been accepted and is merged Semver:minor PR will force a minor update in repo and includes new features and backward compatibility Type:Medium PR is accepted with difficulty level as medium
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants