Skip to content

bug fixes#109

Merged
perib merged 1 commit intoEpistasisLab:devfrom
perib:dev
Nov 3, 2023
Merged

bug fixes#109
perib merged 1 commit intoEpistasisLab:devfrom
perib:dev

Conversation

@perib
Copy link
Collaborator

@perib perib commented Nov 3, 2023

fixed an issue where the old_params in the individual's NodeLabel were not being reset when a new method was selected. Previous solution tried to update the parameters of the old node. This version creates an entirely new node and replaces the old node in the graph.

Also fixed an issue where we could not set both inner_config_dict and leaf_config_dict to None. When mutate_insert_leaf tried to add a leaf, an error was thrown. Now this mutation function is not a possible option unless one of the inner or leaf config dictionaries is set.

I tested this with a configuration dictionary that includes only xgboost and sgdcclassifier since they share a parameter name with different types, which would cause errors in the hyperparamter update function.

from xgboost import XGBClassifier
from sklearn.linear_model import SGDClassifier
from tpot2.config.classifiers import params_SGDClassifier, params_XGBClassifier

root_config_dict =  {SGDClassifier : params_SGDClassifier,
                     XGBClassifier : params_XGBClassifier,}

@perib perib marked this pull request as ready for review November 3, 2023 19:18
@perib perib merged commit f298771 into EpistasisLab:dev Nov 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant