Skip to content
This repository has been archived by the owner on Apr 8, 2024. It is now read-only.

Implement common data loader to support csv/libsvm in all inferencing #28

Open
jfomhover opened this issue Aug 27, 2021 · 1 comment
Open
Labels
bug Something isn't working inferencing-benchmark
Milestone

Comments

@jfomhover
Copy link
Contributor

Lightgbm_python and treelite_python both consume inference data.

That data could be either csv or libsvm, but because we're using numpy in treelite_python, it doesn't support libsvm.

To avoid writing the same data loading code twice, we want to implement a common class for handling that through either numpy or libsvm or lightgbm itself.

@jfomhover jfomhover added this to the Foundations milestone milestone Aug 27, 2021
@jfomhover jfomhover added the bug Something isn't working label Aug 27, 2021
@jfomhover
Copy link
Contributor Author

This is less relevant now that the LightGBM C API uses the LightGBM Parser #153

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working inferencing-benchmark
Projects
None yet
Development

No branches or pull requests

1 participant