or-fusion / pao

A Python Package for Adversarial Optimization
Other
18 stars 15 forks source link

Missing parameters in function call check_termination in pccg_solver.py #97

Open KuKi2121 opened 1 year ago

KuKi2121 commented 1 year ago

Dear pao-Team,

thank you for the development of this amazing package! :) I would like to report a minor bug in the function check_termination in pccg_solver.py. If execute_PCCG_solver reaches its iteration-limit and the terminition is checked, while the solver config.quiet-parameter is set True, check_termination is called and raises the RuntimeError(f'Error: Upper bound greater than lower bound after {k} iterations and {elapsed} seconds: Obj={LB} UB={UB}'). However, this is resulting in a NameError as k (and elapsed) is not provided to the check_termination-function. If PRs are considered, I would be happy to solve this Issue.

Thank you in advance! Best, Kirill Kuroptev