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.

Sign up for GitHub

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

Jump to bottom

Remove distributors from dict cause hangs on GUI #186

Closed
hildogjr opened this issue Mar 5, 2018 · 0 comments
Closed

Remove distributors from dict cause hangs on GUI #186

hildogjr opened this issue Mar 5, 2018 · 0 comments
Labels
bug Bugs that impacts on main KiCost functionality.

Comments

@hildogjr
Copy link
Owner

hildogjr commented Mar 5, 2018 •
edited
Loading

When I run KiCost from the GUI, it call the kicost() ( last version to avoid other process and in next release fix #144).
But, if I ask to not scrape some distributor (and because it, this is removed from distributor_dict)m the next call from the GUI, without restart, hangs KiCost.

Way to solution: not pop the distributor_dict, instead use distributor_dict[dist_list] in the loops. dist_list is a kicost() input parameter.
@hildogjr hildogjr added the bug Bugs that impacts on main KiCost functionality. label Mar 5, 2018
@hildogjr hildogjr changed the title Removed distributors from dict cause hangs on GUI Remove distributors from dict cause hangs on GUI Mar 5, 2018
@hildogjr hildogjr mentioned this issue Jun 5, 2018
Closed
@mmmaisel mmmaisel mentioned this issue Jun 15, 2018
Merged
hildogjr added a commit that referenced this issue Jun 15, 2018
@hildogjr
Dynamically initialize distributor_dict & fix #186
56e1504 
Dynamically initialize distributor_dict.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Bugs that impacts on main KiCost functionality.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@hildogjr