openvino icon indicating copy to clipboard operation
openvino copied to clipboard

[Good First Issue]: Get rid of hardcoded properties name such as "EXPORT_IMPORT" from Python tests

Open akuporos opened this issue 1 year ago • 3 comments

Context

We have properties instead and should use them instead of hardcoded strings like "EXPORT_IMPORT", "OPTIMIZATION_CAPABILITIES" which are actually implementation details.

What needs to be done?

Search across this folder and find where properties are used like string objects (e.g. "OPTIMIZATION_CAPABILITIES"). OpenVINO Python API has dedicated submodule for properties, so these string should be replaced by objects from openvino.properties.

Example Pull Requests

No response

Resources

Contact points

@akuporos @jiwaszki @p-wysocki

Ticket

132906

akuporos avatar Feb 23 '24 11:02 akuporos

.take

inbasperu avatar Feb 23 '24 12:02 inbasperu

Thank you for looking into this issue! Please let us know if you have any questions or require any help.

github-actions[bot] avatar Feb 23 '24 12:02 github-actions[bot]

@akuporos, I have created a PR addressing the following issue and look forward to your feedback on it: https://github.com/openvinotoolkit/openvino/pull/23063#issue-2151650002.

inbasperu avatar Feb 23 '24 19:02 inbasperu

congrats @inbasperu and thank you for your contribution <3

mlukasze avatar Feb 28 '24 09:02 mlukasze