Skip to content
This repository has been archived by the owner on Feb 18, 2023. It is now read-only.

Feature importance #106

Merged
merged 7 commits into from
May 14, 2021
Merged

Feature importance #106

merged 7 commits into from
May 14, 2021

Conversation

joclement
Copy link
Owner

No description provided.

@joclement joclement self-assigned this May 13, 2021
@joclement joclement linked an issue May 13, 2021 that may be closed by this pull request
Just calculate the feature importance for fingerprint (feature-based)
models and exclude the k-NN classifier as the library does not seem to
support it.
@joclement joclement force-pushed the feature-importance branch 11 times, most recently from 88d019c to a6ccc0d Compare May 14, 2021 11:43
The CI fails due to something of the SHAP dependencies not being
installed correctly. Part of the error output is:
C extension was not built during install!

Fix this by restricting the feature importance calculation to the LGBM
classifier as that classifier does not need the C extension.
@joclement joclement merged commit cce9858 into main May 14, 2021
@joclement joclement deleted the feature-importance branch May 14, 2021 12:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make feature importance plot
1 participant