update to kfactory backend #2766
Conversation
![sourcery-ai[bot]](https://avatars.githubusercontent.com/in/48477?s=60&v=4){kind=small}
There was a problem hiding this comment.
The pull request #2766 has too many files changed.
We can only review pull requests with up to 300 changed files, and this pull request has 566.
github-actions
bot
added
the
documentation
|
I wonder if the potential licensing conflict that may arise from bringing in a tight dependence to GPL-licensed code (KLayout) was analyzed and taken into account here. The author of KLayout interprets there not to be an issue with Python import statements, but it's not really his call since KLayout itself is forced to use GPL due to its dependencies. So it doesn't seem like a negligible risk, or at least it's a significant potential complication, and therefore worth putting some serious thought into before creating a hard-to-reverse dependence on GPL-licensed code. If this is already fully understood and the risk has been accepted, then no worries! |
@sebastian-goeldi