Closed remo-help closed 1 year ago
hold off on review while I add tests
Base: 96.90% // Head: 96.90% // No change to project coverage :thumbsup:
Coverage data is based on head (
bdf6107
) compared to base (bdf6107
). Patch has no changes to coverable lines.:exclamation: Current head bdf6107 differs from pull request most recent head f3d5658. Consider uploading reports for the commit f3d5658 to get more accurate results
:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
This PR adds a simple utility method to the Learner class, allowing it to return the feature names used at the estimator step as a 1-D array. Learners with a selector potentially have a smaller feature name list than is stored in the feature_vectorizer. This method allows for easy access of the actual feature names of features used, even if the selector has removed specific features.