Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add BaseStep.config, BaseStep.get_config() and BaseStep.set_config() #91

Closed
guillaume-chevalier opened this issue Oct 1, 2019 · 1 comment
Labels
enhancement New feature or request wontfix This will not be worked on
Milestone

Comments

@guillaume-chevalier
Copy link
Member

The concept is the same as for the Hyperparameter Samples and the Hyperparameter Spaces. But a config shouldn't change what happens to the data, just how it is treated (e.g.: number of cores).

It's interesting to move some parameters to a config for when those parameters are system-related or misc. We don't want some of those parameters to alter the hashes (e.g.: n_jobs in FeatureUnion shouldn't change the outcome and should be modified to be such a config parameter).

@stale
Copy link

stale bot commented Jun 28, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs in the next 180 days. Thank you for your contributions.

@stale stale bot added the wontfix This will not be worked on label Jun 28, 2021
guillaume-chevalier added a commit to Kimoby/Neuraxle that referenced this issue Oct 28, 2021
guillaume-chevalier added a commit to Kimoby/Neuraxle that referenced this issue Oct 28, 2021
@stale stale bot closed this as completed Dec 25, 2021
@guillaume-chevalier guillaume-chevalier added this to the 0.7.0 milestone Apr 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

1 participant