Different approach to wrapping #24
SageCreations
started this conversation in
Ideas
Replies: 1 comment 4 replies
-
I like your fork, and yes, feel free to merge your fork into the python-webui repo. PR: SageCreations#2 |
Beta Was this translation helpful? Give feedback.
4 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
@AlbertShown , I just noticed you updated the wrapper. Thanks for that as I'm trying to push for it's use in a project at my work. I've forked the project and was experimenting with a different approach that might be easier for maintainability if you want to take a look. I was targeting 2.5.0.beta-2 of the C version and heavily documenting it all so that IDE's can explain functions better.
my fork
I've separated all the c-python in one file, and left only the python wrapper functions/classes in the webui.py file. The library loading functions is in its own script and is only used in the c-python file. This makes most of the wrapper functions just a couple or even single lines long.
Beta Was this translation helpful? Give feedback.
All reactions