This repository was archived by the owner on Nov 16, 2023. It is now read-only.
Add get_feature_contributions method to Pipeline - #186
Closed
najeeb-kazmi wants to merge 32 commits into
Closed
Conversation
ganik
reviewed
Jul 15, 2019
ganik
reviewed
Jul 15, 2019
ganik
reviewed
Jul 15, 2019
ganik
reviewed
Jul 15, 2019
ganik
reviewed
Jul 15, 2019
ganik
reviewed
Jul 15, 2019
najeeb-kazmi
changed the title
ganik
reviewed
Jul 18, 2019
ganik
reviewed
Jul 18, 2019
ganik
reviewed
Jul 18, 2019
ganik
reviewed
Jul 18, 2019
ganik
reviewed
Jul 18, 2019
ganik
reviewed
Jul 18, 2019
* Save the model file when pickling a NimbusML Pipeline. * Add version to the pickled Pipeline. * Add the steps attribute to a pickled Pipeline instance. * Add extra unit test for pickled nimbusml pipelines. * Add export_version to pickled base_pipeline_items. Remove unnecessary export_version attribute from an unpickled Pipeline.
* Remove stored references to X and y in BasePredictor. * Remove unnecessary scikit-learn import.
ganik
reviewed
Jul 19, 2019
ganik
reviewed
Jul 19, 2019
| "Restore a pickled object." | ||
| for k, v in state.items(): | ||
| if k not in {'modelbytes', 'type'}: | ||
| if k not in {'modelbytes', 'type', 'export_version'}: |
Contributor
There was a problem hiding this comment.
can u rebase ? this change should be in master already
Member
Author
There was a problem hiding this comment.
I did, but the commit history in this branch got screwed up while fixing conflicts. This (and other changes) aren't actually in this PR.
Member
Author
|
Closing this and replacing with PR #196 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fix #91
Adds a
get_feature_contributions()method toPipeline, which works similarly topredict()but adds atransforms_featurecontributioncalculationtransformernode after thetransforms_datasetscorernode in the graph that gets executed.Adds a
get_feature_contributions()method toBasePredictorso that models trained outside of aPipelinealso return feature contributions.Adds an example of how to use this.
Adds a test to check that an unpickled model can calculate feature contributions.
Adds a test to check that a model loaded from zip can calculate feature contributions.