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

Influence.py __init___ & run_influence merge? #138

Open
JGASmits opened this issue Sep 29, 2021 · 0 comments
Open

Influence.py __init___ & run_influence merge? #138

JGASmits opened this issue Sep 29, 2021 · 0 comments

Comments

@JGASmits
Copy link
Contributor

The influence command functions are scattered between the init part of the influence.py and the .run_influence command. Perhaps this is needed for some multi-threading that i'm not aware of, in that case feel free to close this issue.

But I think it would greatly increase understandability of the code if everything is bellow the init instead of spread to two different functions (with the function defenitions also being spread out over regular functions and functions part of the class object).

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

No branches or pull requests

1 participant