openvino icon indicating copy to clipboard operation
openvino copied to clipboard

Published 20 hours ago • verified publisher icon openvinotoolkit

Force default locale for cache model write/read

Open zhaixuejun1993 opened this issue 9 months ago • 2 comments

Details:

  • Check the value of setlocale for export/import, if different with "C" will set to "C" and record the original value, after export/import done will reset to the original.
  • Fix the error caused by pugixml library with The setlocale function installs the specified system locale or its portion as the new C locale. different C may return unexpected results with setlocal()

Tickets:

  • https://github.com/openvinotoolkit/openvino/issues/24370

zhaixuejun1993 avatar May 09 '24 07:05 zhaixuejun1993

@ilya-lavrenov could you check whether this fixing is reasonable? Thanks!

riverlijunjie avatar May 09 '24 09:05 riverlijunjie

@RyanMetcalfeInt8 Could you please check whether the fix helps in your case?

ilya-lavrenov avatar May 11 '24 14:05 ilya-lavrenov

@RyanMetcalfeInt8 Could you please check whether the fix helps in your case?

Do I need to build OpenVINO in order to try this, or can I grab a CI build?

RyanMetcalfeInt8 avatar May 13 '24 13:05 RyanMetcalfeInt8

@RyanMetcalfeInt8 Could you please check whether the fix helps in your case?

Do I need to build OpenVINO in order to try this, or can I grab a CI build?

The CI build is enough, I think.

zhaixuejun1993 avatar May 14 '24 01:05 zhaixuejun1993

@RyanMetcalfeInt8 Could you please check whether the fix helps in your case?

Do I need to build OpenVINO in order to try this, or can I grab a CI build?

The CI build is enough, I think.

Can you point me to where I can download the CI build? I was trying to find a link to artifacts from GitHub actions, but I couldn't seem to find it..

RyanMetcalfeInt8 avatar May 14 '24 01:05 RyanMetcalfeInt8

@RyanMetcalfeInt8 Could you please check whether the fix helps in your case?

Do I need to build OpenVINO in order to try this, or can I grab a CI build?

The CI build is enough, I think.

Can you point me to where I can download the CI build? I was trying to find a link to artifacts from GitHub actions, but I couldn't seem to find it..

http://ov-share-03.iotg.sclab.intel.com/volatile/openvino_ci/private_builds/dldt/master/pre_commit/0924c59be2edba83c088890a5a03e5b3712b2e15/private_linux_ubuntu_20_04_release/cpack/

zhaixuejun1993 avatar May 14 '24 01:05 zhaixuejun1993

@RyanMetcalfeInt8 Could you please check whether the fix helps in your case?

Do I need to build OpenVINO in order to try this, or can I grab a CI build?

The CI build is enough, I think.

Can you point me to where I can download the CI build? I was trying to find a link to artifacts from GitHub actions, but I couldn't seem to find it..

image

ilya-lavrenov avatar May 15 '24 10:05 ilya-lavrenov

@RyanMetcalfeInt8 Could you please check whether the fix helps in your case?

Do I need to build OpenVINO in order to try this, or can I grab a CI build?

The CI build is enough, I think.

Can you point me to where I can download the CI build? I was trying to find a link to artifacts from GitHub actions, but I couldn't seem to find it..

image

Thanks, I have verified that this PR resolves the locale issue.

RyanMetcalfeInt8 avatar May 15 '24 23:05 RyanMetcalfeInt8