You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
As a user, I want to ensure that the solvers will also run in case that the Process decorators are integrated differently in the future. But at the moment, the builder.py replaces decorators in this non-future-proof way: setattr(solver_model, "implements_process", self.solver_process) # Get requirements of parent class super_res = solver_model.required_resources.copy() # Set new requirements not overwriting parent class requirements. setattr(solver_model, "required_resources", super_res + requirements) setattr(solver_model, "implements_protocol", protocol)
Conditions of satisfaction
Above-mentioned code should use the clean interface developed as solution to issue #580
The text was updated successfully, but these errors were encountered:
User story
As a user, I want to ensure that the solvers will also run in case that the Process decorators are integrated differently in the future. But at the moment, the builder.py replaces decorators in this non-future-proof way:
setattr(solver_model, "implements_process", self.solver_process) # Get requirements of parent class
super_res = solver_model.required_resources.copy() # Set new requirements not overwriting parent class requirements.
setattr(solver_model, "required_resources", super_res + requirements)setattr(solver_model, "implements_protocol", protocol)
Conditions of satisfaction
The text was updated successfully, but these errors were encountered: