Seung Hyun Kim

**Comment by [armantekinalp](https://github.com/armantekinalp)** _Thursday Dec 30, 2021 at 05:29 GMT_ ---- In future implementation of restart I think we should add these functionalities: - Current implementation allows only loading systems...

@Anika-Roy Thank you for your interest. Yes, this issue is still open. If you are interested in working, let me know, I can assign you.

> @skim0119 @armantekinalp @bhosale2 Do you have an update on reviewing the proposed mixin and interface? @mstoelzle Hi -- Sorry for taking some time to answer your question. Our original...

@mstoelzle Hi -- Thank you for your patience. We have some criteria we would like to include on the controller. Combined with your example script above and your question on...

@mstoelzle Thank you for your response. I'll try to make some comments and answer some questions. ## On Major Comments, I'm not quite sure if I understood your example case...

> ### Regarding actions > Ok thanks, I understand the purpose a bit better then. > > > parameters = [np.ones(17,), np.zeros(17,)] > > controller.set_action(parameters) > > I don't understand...

@mstoelzle -- Something along this line? ```py class ModelProtocol(Protocol): def predict(self, X: np.ndarray) -> np.ndarray: ... class SerialManipulatorController(ControllerBase): def __init__(self, model: ModelProtocol, *args, **kwargs): super().__init__(*args, **kwargs) self.model = model self.z...

@mstoelzle > I still think it is a bit overly complicated and could be implemented in a lighter way. Our focus is more geared towards delivering a "generalizable" solver, and...

**Comment by [tp5uiuc](https://github.com/tp5uiuc)** _Sunday Dec 12, 2021 at 19:10 GMT_ ---- It would be good if figure out the Controller interface here. For the Environments, you can take a look...

I think `collision` should be separated. I agree that `connection could be more general name.