probatus icon indicating copy to clipboard operation
probatus copied to clipboard

Investigate if parts of the codebase can leverage other libraries code

Open ReinierKoops opened this issue 2 years ago • 2 comments

Problem Description We have some custom implementations of pretty basic functionality code which is often done better by other libraries that are already part of the required libraries to run Probatus. Maybe this would allow us to clean up more code and use more optimized functions instead.

Desired Outcome Replace functions which are better done in the respective libraries as compared to some of the custom implementations in Probatus

Solution Outline Investigate code, especially code which hasn't been updated recently. Find which functions can be replaced and do so.

ReinierKoops avatar Mar 18 '24 20:03 ReinierKoops

@PaulZhutovsky The scoring.py allows us to define multiple scorers. This might be an interesting use-case. I am not certain yet if this is something we want to expand on or not. Maybe something to consider before thinking of replacing it with a simple sklearn implementation because that would inadvertently change it so just a single scorer.

ReinierKoops avatar Mar 20 '24 18:03 ReinierKoops

After another investigation, the Probatus implementation seems the most elegant for getting the name of the scorer since Sklearn does not provide that functionality. It only allows us to get the names of all possible scorers, but not the name given a scorer. @PaulZhutovsky

ReinierKoops avatar Mar 28 '24 09:03 ReinierKoops