search

Moffran / calibrated_explanations

Repository for the explanation method Calibrated Explanations (CE)
BSD 3-Clause "New" or "Revised" License
39 stars 5 forks source link

Restructure the `CalibratedExplanations` class #14

Closed tuvelofstrom closed 10 months ago

tuvelofstrom commented 10 months ago

Is your feature request related to a problem? Please describe. The CalibratedExplanations currently provide functionality for a mix between a set of explanations and individual explanations.

Describe the solution you'd like I would like to separate between the set of CalibratedExplanations and individual CalibratedExplanation in a better way.

Describe alternatives you've considered It would make sense to create subclasses for the CalibratedExplanation representing factual and counterfactual explanations, as these clearly have class specific behaviour. It would also make sense to create subclasses representing regression vs probabilistic explanations, as they differ in regards to the plots. However, would it make sense to create e.g. FactualRegression, CounterfactualRegression, FactualProbabilistic and CounterfactualProbabilistic?

tuvelofstrom commented 10 months ago

With 1e080c65d118f063962b2dff651942ec3ea7bd37, this issue is resolved.