LSSTDESC / rail_sklearn

RAIL algorithms that depend on scikit-learn.
MIT License
1 stars 0 forks source link

Add random forest classifier for tomographer #1

Closed hangqianjun closed 1 year ago

hangqianjun commented 1 year ago

Change Description

Solution Description

Moved random forest classifier for the tomographer class from rail_base to rail_sklearn due to its dependence on sklearn.

Code Quality

Project-Specific Pull Request Checklists

Bug Fix Checklist

New Feature Checklist

Documentation Change Checklist

Build/CI Change Checklist

Other Change Checklist

codecov[bot] commented 1 year ago

Welcome to Codecov :tada:

Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.

Thanks for integrating Codecov - We've got you covered :open_umbrella:

hangqianjun commented 1 year ago

Unit tests pass on local machine but not on GitHub. Waiting for new release of rail to include classifier class.