Closed ashishpatel16 closed 5 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 96.87%. Comparing base (
4595264
) to head (5aa9998
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Hi @mirand863, this is the final version of the Explainer API for LCPN and LCPPN models. LCPL will follow soon.
The changes made here are as follows -
I'm looking forward to your remarks. Hoping this will be the finally merged!
Hi @mirand863, I have incorporated the following changes -
Hi @ashishpatel16
The colabs notebook does not work unless I update the git checkout to git checkout --track origin/lcpn_explainer
Can you please update it?
Hi @mirand863, Thanks for the review. I have updated the Colab notebook with the requested changes. Please check it and let me know if it works perfectly now! ✅
Hi @mirand863, Thanks for the review. I have updated the Colab notebook with the requested changes. Please check it and let me know if it works perfectly now! ✅
It works now, Thanks!
This PR aims to add the implementation for LCPN explainer. The major changes in the code are as follows -
explain_with_xr
method relies onget_traversed_nodes
for accessing the correct nodes for traversing the local classifiers, changes are only required in theget_traversed_nodes
function to return traversed nodes depending on the Hiclass model provided.lcpn.predict()
) represents the node traversed at each hierarchy level. This is implemented in the corresponding method.