Change default budget in AADForest to "auto"#317
Conversation
There was a problem hiding this comment.
Pull request overview
This pull request changes the default value of the budget parameter in the AADForest class from 0.03 to "auto" to reduce the number of hyperparameters users need to configure.
Changes:
- Updated default
budgetparameter from0.03to"auto"inAADForest.__init__() - Updated parameter documentation to reflect the new default value
- Added explicit
budget=0.03to a regression test
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| src/coniferest/aadforest.py | Changed default budget parameter from 0.03 to "auto" in both the method signature and documentation |
| tests/test_aadforest.py | Added explicit budget=0.03 parameter to test_regression_fit_known |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
8c72dcd to
0dcdc6d
Compare
As was previously discussed, this helps to reduce number of hyperparameters.
for more information, see https://pre-commit.ci
Merging this PR will improve performance by ×18
Performance Changes
Comparing Footnotes
|
This is a follow up to the latest meeting. As was previously discussed, this helps to reduce number of hyper-parameters.