-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[pythonscripting][wip] initial release #18229
base: main
Are you sure you want to change the base?
Conversation
Working with "info is null" Working - occasional "info" is null
Graal engine working - occasional "info" is null
Working with "info is null" Working - occasional "info" is null
added "py" extension in getExtension First pass at mirroring jsscripting Created own GraalPythonScriptEngine based on the GraalJSScriptEngine Working - with exception of regex language updated working/lib directories to find modules in lib folder added python standard libraries Working version
Signed-off-by: Holger Hees <[email protected]>
added "py" extension to getExtension in factory
Signed-off-by: Holger Hees <[email protected]>
Added injection of scope hashmap
inserted scope variables from the scopeValues method in the component
… - not needed since one is created in OpenhabGraalPythonScriptEngine cleanup
Eliminate creating a default engine in GraalPythonScriptEngineFactory…
Added setItemMetadata function
…fDifferent for openhab value types like UP/DOWN etc)
FYI, I plan to use GraalPython with the MQTT Home Assistant binding so that I can run actual Jinja, and not Jinjava which has had significant compatibility issues with more complex templates that devices can provide. We should probably avoid having multiple copies of the GraalPython bundles loaded, so I've started work on OSGi-ifying the Python specific bundles here. You can reference #18053 for how to begin depending on the OSGi-ified versions of bundles. |
Initial release with detailed description in marketplace
https://community.openhab.org/t/python3-scripting/161945
and it has still one major known issues #18054 (comment)
in the final version, before I merge, I will rebase the project to clean the history.