change estimators to 1250

[peterdudfield]

Pull Request

Description

Increase n estimators to 1250, trained and saved new models to v3 Also did small ablation study

How Has This Been Tested?

Ci tests

• [x] Yes

Checklist:

• [ ] My code follows OCF's coding style guidelines
• [x] I have performed a self-review of my own code
• [ ] I have made corresponding changes to the documentation
• [ ] I have added tests that prove my fix is effective or that my feature works
• [ ] I have checked my code and corrected any misspellings

[codecov[bot]]

Codecov Report

Merging #53 (a428b08) into main (cfc2c97) will not change coverage. The diff coverage is 50.00%.

@@           Coverage Diff           @@
##             main      #53   +/-   ##
=======================================
  Coverage   43.05%   43.05%           
=======================================
  Files          21       21           
  Lines         950      950           
=======================================
  Hits          409      409           
  Misses        541      541           

Impacted Files	Coverage Δ
gradboost_pv/models/s3.py	0.00% <0.00%> (ø)
gradboost_pv/models/training.py	0.00% <ø> (ø)
gradboost_pv/__init__.py	100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more