Skip to content

Conversation

@shubskmr
Copy link

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>
@shubskmr
Copy link
Author

@nrspruit @rwmcguir can you please take a look and review this PR?

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>
@shubskmr
Copy link
Author

shubskmr commented Feb 9, 2026

@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?

@shubskmr shubskmr requested a review from rwmcguir February 10, 2026 04:08
@@ -0,0 +1,7 @@
"""
Copy link
Contributor

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
Copy link
Contributor

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
Copy link
Contributor

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

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

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.

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),

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),

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 ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants