-
Notifications
You must be signed in to change notification settings - Fork 123
feature: Add support for sysman python bindings #420
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
base: master
Are you sure you want to change the base?
Conversation
Related-To: NEO-NEO-17257 Signed-off-by: shubham kumar <shubham.kumar@intel.com>
Related-To: NEO-NEO-17257 Signed-off-by: shubham kumar <shubham.kumar@intel.com>
Related-To: NEO-NEO-17257 Signed-off-by: shubham kumar <shubham.kumar@intel.com>
Related-To: NEO-NEO-17257 Signed-off-by: shubham kumar <shubham.kumar@intel.com>
Signed-off-by: shubham kumar <shubham.kumar@intel.com>
Related-To: NEO-NEO-17257 Signed-off-by: shubham kumar <shubham.kumar@intel.com>
Related-To: NEO-NEO-17257 Signed-off-by: shubham kumar <shubham.kumar@intel.com>
Signed-off-by: shubham kumar <shubham.kumar@intel.com>
Signed-off-by: shubham kumar <shubham.kumar@intel.com>
Signed-off-by: shubham kumar <shubham.kumar@intel.com>
|
@rwmcguir I have added the workflow for bindings ULT coverage along with the .toml file for package creation. Can you please take a look and review them? |
| @@ -0,0 +1,7 @@ | |||
| """ | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Silly, but this needs Copyright / License as well.
| @@ -0,0 +1,53 @@ | |||
| name: Python Bindings - Security Scan | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If we have the ability to Copyright/License this file, please do if .yml supports comments
| @@ -0,0 +1,300 @@ | |||
| name: Python Bindings - Unit Tests & Coverage | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If we have the ability to Copyright/License this file, please do if .yml supports comments
| @@ -0,0 +1,105 @@ | |||
| ## | |||
| # Copyright (C) 2025 Intel Corporation | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please update copyrights
| @@ -0,0 +1,1030 @@ | |||
| #!/usr/bin/env python3 | |||
| ## | |||
| # Copyright (C) 2025 Intel Corporation | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copyrights in all files could be updated
|
|
||
| class _PrintableStructure(Structure): | ||
| """ | ||
| Abstract class that produces nicer __str__ output than ctypes.Structure. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need this comments ?
| ("timerResolution", c_uint64), | ||
| ("timestampValidBits", c_uint32), | ||
| ("kernelTimestampValidBits", c_uint32), | ||
| ("uuid", ze_device_uuid_t), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this ze_device_uuid_t or zes counter part ?
| ("type", ze_device_type_t), | ||
| ("vendorId", c_uint32), | ||
| ("deviceId", c_uint32), | ||
| ("flags", ze_device_property_flags_t), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a zes counterpart ?
Related-To: NEO-NEO-17257
Signed-off-by: shubham kumar shubham.kumar@intel.com