Skip to content

Fix: Handle missing time indices gracefully for time-based features (… - #2764

Open
realranjan wants to merge 1 commit into
alteryx:mainfrom
realranjan:fix-issue-2700
Open

Fix: Handle missing time indices gracefully for time-based features (…#2764
realranjan wants to merge 1 commit into
alteryx:mainfrom
realranjan:fix-issue-2700

Conversation

@realranjan

Copy link
Copy Markdown
 ## What does this PR do?
 - Fixes [Issue #2700](https://github.com/alteryx/featuretools/issues/2700): Previously, if any row in a DataFrame had a missing time index, all features for that row became NaN.
 - Now, only time-based features are set to NaN for rows with missing time index; non-time-based features are still computed.
 - Adds/updates tests (including `test_missing_time_index_rows`) to verify correct behavior.

 ## How was this tested?
 - All tests pass locally.
 - The new test ensures only time-based features are NaN for missing time index rows, while non-time-based features are always computed.

 ## Checklist
 - [x] Fix implemented
 - [x] Tests updated/added
 - [x] All tests passing
 - [x] Linked to #2700

 Closes #2700

Screenshot 2025-06-09 193132

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.

1 participant