skll icon indicating copy to clipboard operation
skll copied to clipboard

Added a utility method to the Learner class returning feature names

Open remo-help opened this issue 3 years ago • 2 comments

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.

remo-help avatar Jun 28 '22 19:06 remo-help

hold off on review while I add tests

remo-help avatar Jun 28 '22 19:06 remo-help

Codecov Report

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

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #715   +/-   ##
=======================================
  Coverage   96.90%   96.90%           
=======================================
  Files          63       63           
  Lines        9271     9271           
=======================================
  Hits         8984     8984           
  Misses        287      287           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

codecov[bot] avatar Jun 28 '22 19:06 codecov[bot]