Skip to content

feat: W2D1 Feature Engineering & Encoding - #1863

Closed
krishnatejanagarajupalli1976-debug wants to merge 2 commits into
Data-Centric-AI-Community:developfrom
krishnatejanagarajupalli1976-debug:feat/aiml-W2-krishnateja
Closed

krishnatejanagarajupalli1976-debug wants to merge 2 commits into
Data-Centric-AI-Community:developfrom
krishnatejanagarajupalli1976-debug:feat/aiml-W2-krishnateja

Conversation

@krishnatejanagarajupalli1976-debug

Copy link
Copy Markdown

W2D1: Feature Engineering & Encoding

Implemented feature engineering and preprocessing techniques:

  • LabelEncoder
  • OneHotEncoder
  • OrdinalEncoder
  • StandardScaler
  • MinMaxScaler
  • RobustScaler
  • Before/after scaling distribution plots
  • SelectKBest for top 5 feature selection
  • Feature leakage considerations

Branch: feat/aiml-W2-krishnateja

@krishnatejanagarajupalli1976-debug krishnatejanagarajupalli1976-debug changed the title feat: W2D1 Feature Engineering & Encodingkrishnateja feat: W2D1 Feature Engineering & Encoding Sep 10, 2026
@fabclmnt
fabclmnt self-requested a review September 11, 2026 17:00
@fabclmnt

Copy link
Copy Markdown
Collaborator

Thanks for taking the time to open this.

This doesn't fit the scope of the project. fg-data-profiling is an EDA
library: it describes a dataset and surfaces its problems. Feature
engineering and preprocessing like encoders, scalers, SelectKBest, are
downstream modelling steps that belong in a scikit-learn pipeline, not here.

Scope aside, the notebook is standalone: it sits at the repo root, never
imports the package, and runs on a generated toy dataset, so it doesn't
add to the library, the docs, or the test suite.

The branch name suggests this was meant for a course repository rather
than this one. Closing for now. If you do want to contribute, the open
issues and CONTRIBUTING.md are the place to start.

@fabclmnt fabclmnt closed this Sep 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants