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

Wrap config.py config.config_dict in a class so values don't change #32

Open
tnigon opened this issue May 14, 2020 · 0 comments
Open

Wrap config.py config.config_dict in a class so values don't change #32

tnigon opened this issue May 14, 2020 · 0 comments
Labels
bug Something isn't working

Comments

@tnigon
Copy link
Contributor

tnigon commented May 14, 2020

The config_dict changes when used over and over (passed to various class instances - FeatureSelection, Training, etc.), and when the variables are used, they are being modified in the original file. I try to use dict.copy() anytime it's used, but this is a soft copy and modifies the config.

copy.deepcopy() works, but requires that another library is imported.

To do

Decide whether to go forward with deepcopy or to encapsulate config_dict (and perhaps other settings) in a class.

@tnigon tnigon added the bug Something isn't working label May 14, 2020
@tnigon tnigon self-assigned this May 14, 2020
@tnigon tnigon removed their assignment Oct 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant