Skip to content

Fix ONSAN-2069#2619

Closed
jafraustro wants to merge 1 commit intooneapi-src:developmentfrom
jafraustro:Getting_Started_ITEX_final
Closed

Fix ONSAN-2069#2619
jafraustro wants to merge 1 commit intooneapi-src:developmentfrom
jafraustro:Getting_Started_ITEX_final

Conversation

@jafraustro
Copy link

Existing Sample Changes

Description

Install dependencies from conda forge. pip install -r requirements.txt command fetchs tensorflow 2.18.
The script infer_resnet50.py works correctly.

There still a problem with oneDNNLog( ), but is out of the scope for this issue

Type of change

Please delete options that are not relevant. Add a 'X' to the one that is applicable.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Implement fixes for ONSAM Jiras

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • Command Line
  • oneapi-cli
  • Visual Studio
  • Eclipse IDE
  • VSCode
  • When compiling the compliler flag "-Wall -Wformat-security -Werror=format-security" was used

@jafraustro jafraustro mentioned this pull request Mar 7, 2025
9 tasks
Install dependencies from conda forge. `pip install -r requirements.txt` command fetches tensorflow 2.18. The script `infer_resnet50.py` works correctly.
Ticket: ONSAN-2069

Co-authored-by:  Manuel Santana <manuel.santana.castolo@intel.com>
Signed-off-by: Manuel Santana <manuel.santana.castolo@intel.com>
Signed-off-by: jafraustro <jaime.fraustro.valdez@intel.com>
@Ankur-singh
Copy link
Contributor

@jafraustro thank you for submitting the PR!

The code sample is quite involved as it tries to test the sample in both tensorflow and tensorflow-gpu venvs. Also, it does a few snoopy things like:

source $ONEAPI_INSTALL/setvars.sh --force > /dev/null 2>&1
source activate user-tensorflow
echo "########## Executing the run"
DNNL_VERBOSE=1 python infer_resnet50.py > infer_rn50_cpu.csv
echo "########## Done with the run"

@jimmytwei will this sample be moved out from this repo? If not, then we should have a major re-write.

@jimmytwei
Copy link
Contributor

@Ankur-singh I believe ITEX will be removed in a future AI Tools release so it's not worth fixing this sample as it will eventually be removed from the repo so closing this PR.

@jimmytwei jimmytwei closed this Mar 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants