eyadgaran / SimpleML

Machine learning that just works, for effortless production applications
BSD 3-Clause "New" or "Revised" License
17 stars 5 forks source link

Orm separation #99

Closed eyadgaran closed 2 years ago

eyadgaran commented 2 years ago

Separate out ORM behavior from core persistables. (unlocks potential for swappable backend - e.g. SimpleML-MLFlow)

coveralls commented 2 years ago

Pull Request Test Coverage Report for Build 2216974467


Changes Missing Coverage Covered Lines Changed/Added Lines %
simpleml/orm/dataset.py 8 9 88.89%
simpleml/orm/model.py 10 11 90.91%
simpleml/orm/pipeline.py 9 10 90.0%
simpleml/pipelines/sklearn/base.py 0 1 0.0%
simpleml/models/base_sklearn_model.py 0 2 0.0%
simpleml/orm/base_sqlalchemy.py 10 12 83.33%
simpleml/persistables/base_persistable.py 42 45 93.33%
simpleml/utils/scoring/load_persistable.py 4 7 57.14%
simpleml/orm/metric.py 15 21 71.43%
simpleml/orm/persistable.py 42 48 87.5%
<!-- Total: 236 348 67.82% -->
Files with Coverage Reduction New Missed Lines %
simpleml/persistables/base_persistable.py 1 76.02%
simpleml/pipelines/base_pipeline.py 1 42.86%
simpleml/models/base_model.py 2 26.09%
<!-- Total: 4 -->
Totals Coverage Status
Change from base Build 2216632636: 0%
Covered Lines: 2751
Relevant Lines: 4629

💛 - Coveralls