Skip to content
This repository has been archived by the owner on Jan 9, 2024. It is now read-only.

Autohyperparam #153

Draft
wants to merge 45 commits into
base: development
Choose a base branch
from
Draft

Autohyperparam #153

wants to merge 45 commits into from

Conversation

cchoquette
Copy link
Contributor

Description

Uses hyperparam to enable automatic param distribution for foreshadow object.

adithyabsk and others added 30 commits August 7, 2019 17:25
1. extending the _get_param_names and fixing the class to be parents in the wrapper.
2. Fixing PreparerStep to traverse __mro__.
3. Double checking major classes to ensure get_params returns required values.
Tests included.
…tners/foreshadow into issue_128_hyperparam

# Conflicts:
#	foreshadow/smart/smart.py

updating poetry
…s for our use case. Using patchy to implement this. Switching all imports to use our internal BaseEstimator and TransformerMixin. Adding Smart test.
# Conflicts:
#	foreshadow/base.py
IMPORTANT: cHanges to preparerstep and to DataPreparer to enable from initialization from set_params.
IMPORTANT: Changes to preparerstep and to DataPreparer to enable from initialization from set_params.
@cchoquette cchoquette requested a review from jzhang-gp August 23, 2019 20:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants