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

Numerical and Categorical Pre-Processing PR #29

Closed
wants to merge 9 commits into from

Conversation

Lycons-rage
Copy link
Contributor

Closes: #18 [FEATURE REQUEST] Adding Pre-Processing Techniques for numerical as well as categorical features

  • Title : Standardization, Normalization, Ordinal Encoding Complete
  • Name: Chinmaya Tewari (Lycons-rage)
  • Idenitfy yourself: SSOC Contributor

Describe the add-ons or changes you've made 📃

Added 3 directories, named, Min_Max_Scaler (Normalization), Standard_Scaler (Standardization), Ordinal_Encoding (Categorical Feature Handling), each having testing and images subdirectories along with the python script and testing files and respective readme.md files.

Type of change ☑️

What sort of change have you made:

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Code style update (formatting, local variables)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested? ⚙️

Checklist: ☑️

  • My code follows the Contributing Guidelines & Code of Conduct of this project.
  • This PR does not contain plagiarized content.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly wherever it was hard to understand.
  • My changes generate no new warnings.

Screenshots 📷

Screenshot 2024-06-10 130158

Copy link

Thank you for submitting your pull request! We'll review it as soon as possible. For further communication, join our discord server https://discord.gg/tSqtvHUJzE.

Copy link
Owner

@Avdhesh-Varshney Avdhesh-Varshney left a comment

Choose a reason for hiding this comment

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

@Lycons-rage LGTM. do these necessary changes.
Also enlist these algorithms in the readme file created by me.

The class includes basic exception handling:
- Raises an exception if input data contains null values.
- Raises an exception if data type is not a pandas DataFrame or NumPy array.
- Prints an error message if transformation is attempted before fitting the scaler.
Copy link
Owner

Choose a reason for hiding this comment

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

can you explain this line?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Should I elaborate the line in readme.md file?

Copy link
Owner

Choose a reason for hiding this comment

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

no explain it only in chat only last line.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The last line says that if a user tries to call transform function directly, without either calling fit function or fit_transform function first, it will return an exception as fit function calculates the required parameters on the dataset, which, in this case (Min_Max_Scaler) are the minimum and maximum values of each feature. Without them we cannot transform the data.

Copy link
Owner

Choose a reason for hiding this comment

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

remove this file. not required.

Copy link
Owner

Choose a reason for hiding this comment

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

don't delete any file. revoke it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

don't delete any file. revoke it.

I would've deleted it by mistake...my bad

@Avdhesh-Varshney Avdhesh-Varshney added bug 🐛 Something isn't working Requested Changes ⚙️ Some changes have been requested in this PR labels Jun 10, 2024
@Avdhesh-Varshney
Copy link
Owner

Avdhesh-Varshney commented Jun 12, 2024

@Lycons-rage you have not committed the required change so to reflect on the pr

@Lycons-rage
Copy link
Contributor Author

@Avdhesh-Varshney I'm facing some issues with my laptop, I'll do it by today for sure

@Lycons-rage
Copy link
Contributor Author

@Lycons-rage LGTM. do these necessary changes. Also enlist these algorithms in the readme file created by me.

@Avdhesh-Varshney Could you please tell me in which readme file should i enlist the algorithms

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something isn't working Requested Changes ⚙️ Some changes have been requested in this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEATURE REQUEST] Adding Pre-Processing Techniques for numerical as well as categorical features
2 participants