Closed hcho3 closed 1 year ago
Patch coverage: 86.91
% and project coverage change: +0.23
:tada:
Comparison is base (
0640b75
) 82.86% compared to head (950a814
) 83.09%.:exclamation: Current head 950a814 differs from pull request most recent head 6c511a5. Consider uploading reports for the commit 6c511a5 to get more accurate results
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
@wphicks Can I get a review for this PR?
Clean up the serialization logic using mix-in classes. Closes https://github.com/dmlc/treelite/issues/478. Now all the serialization logic lives in
src/serializer.cc
.I broke up #506 to make it easy to review.