[Enhancement] Separate data preprocessing from plotters #134
Labels
api
Application programming interface
dx
Developer experience
enhancement
Improvement to existing features/functionality
Separate data preprocessing from plotters
Previously proposed in #81 (comment), it might be good to separate data preprocess (could make them private so users could still input any format, make this invisible from user) from plotters, which could hopefully resolve #131 (comment) too.
Suggestions
Currently almost each plotter accept various types of data, but at the cost of plotter being very complex (and repeated code). I would suggest making plotter itself only handle single (or very few) data type and migrate the following data processing to some dedicated utilities:
numpy.array
orpandas.DataFrame
(or some other preferred type)scikit-learn
)Potential Impact
I don't expect this to be breaking (or even visible to user), but certainly would be a lot of work as almost the entire code base need to be refactored.
The text was updated successfully, but these errors were encountered: