NorskRegnesentral / shapr

Explaining the output of machine learning models with more accurately estimated Shapley values
https://norskregnesentral.github.io/shapr/
Other
138 stars 32 forks source link

Expose an option to run the procedure sequentially in explain. #321

Closed jonlachmann closed 1 year ago

jonlachmann commented 1 year ago

For debugging purposes it seemed beneficial to be able to use the non-parallel version of the code. It is also useful if many parallel explanations are run, to avoid over-subscription of the available cores (i.e. running more threads than cores, causing slowdowns).

martinju commented 1 year ago

Thanks! I agree this is useful for devel purposes (which is the reason for putting it in there in the first place). However, I am unsure whether it is wise to add another argument to explain(), as there are already quite many. I will leave this open for now, while thinking it through.

martinju commented 1 year ago

Closing for now as only useful for development (and there are already many arguments in explain())