c3di/im2im

Return the code str or function?

Closed this issue · 0 comments

Currently, the interface returns a tuple of strings, which are not executable Python code without further evaluation. This approach might be intended for dynamically generating code

  • but it could lead to security risks and debugging difficulties. We have unit tests to guarantee its correctly. Alos we provide an interface to accept python code.
  • Consider directly returning the conversion function or applying the conversion within this function. it is ok but involves python dependencies management. It needs some manual operation for denepencies installed like tenforflow-gpu on windows. Also if we provide all dependencies, there is potential conflicts between the packages. Not all of them are used, but they are already installed. Then next reason if the users would like to use our tool in their work, they tendencies already installed, overwise no need to convert