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
Objective of issue: In the repo, Processes are produced on the fly in the SolverProcessBuilder.
That's a feature which should not be hidden in the optimization library, but should be a generic tool in Lava.
In addition, in builder.py, the decorators are currently replaced in this 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)
That will work with the current decorators; but future pushes to the decorators may break it.
Lava version:
0.3.0 (feature release)
The text was updated successfully, but these errors were encountered:
A feature request for something that can generate Lava Processes on the fly. This should be filed in the lava-nc/lava repository.
An idea to make the lava-optimization code more future-proof, which would be a feature request to be filed in lava-nc/lava-optimization. @phstratmann Could you please change the stories accordingly?
Objective of issue: In the repo, Processes are produced on the fly in the SolverProcessBuilder.
That's a feature which should not be hidden in the optimization library, but should be a generic tool in Lava.
In addition, in builder.py, the decorators are currently replaced in this 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)
That will work with the current decorators; but future pushes to the decorators may break it.
Lava version:
The text was updated successfully, but these errors were encountered: