PythonRemoteServer icon indicating copy to clipboard operation
PythonRemoteServer copied to clipboard

Support serving multiple libraries by one server

Open robooo opened this issue 7 years ago • 4 comments

robooo avatar Feb 10 '18 23:02 robooo

We are using this adaptions now for more than one year. It is absolut stable and we recognized no problems. Is there a possibility that this feature will be merged into master in the near future? And if not, are there any doubts about this integration? Can we provide any additional information or help if needed?

Thanks in advance, Mario

loumaran avatar Feb 20 '19 14:02 loumaran

@srinivaschavan @robooo @loumaran would be awesome to see this merged in. Can we get an update here?

@srinivaschavan I'm happy to update documentation if that's what's needed to move this along. That would be primarily the readme in this repo? Perhaps adding a Multiple library support section? Since I don't have write access to the forked repo here, can we merge this then I can create a separate PR to update the docs.

Erich-McMillan avatar Dec 12 '22 21:12 Erich-McMillan

@Erich-McMillan , yes updating the README should suffice. However, the real concern with this PR is that it is not backwards compatible. IMO, we should handle the case where users have not specified a list of libraries.

srinivaschavan avatar Dec 13 '22 04:12 srinivaschavan

@Erich-McMillan , yes updating the README should suffice. However, the real concern with this PR is that it is not backwards compatible. IMO, we should handle the case where users have not specified a list of libraries.

Agreed, best to allow users to continue passing in a single library without putting it into a list. Would the best path forward be to have the constructor check the type of library to see if it is a list then act accordingly?

Also if no feedback from previous authors, I can create a new PR to introduce those changes alongside docs.

Erich-McMillan avatar Dec 13 '22 16:12 Erich-McMillan