-
Notifications
You must be signed in to change notification settings - Fork 211
Conversation
intel_extension_for_transformers/neural_chat/pipeline/plugins/audio/tts.py
Outdated
Show resolved
Hide resolved
intel_extension_for_transformers/neural_chat/pipeline/plugins/audio/tts.py
Outdated
Show resolved
Hide resolved
Yes, we don't need ipex.cpu.runtime neither. We can just set the core binding using numactl -m 0 -C cores to get the similar effect. |
Absolutely. I'm just concerning that running the neuralchat service shell script will normally bind to one socket, which may be different cores than 24. But normally I believe it is slight performance difference. |
As we discussed, there might be no additional perf optimizations on HPU at this moment. Can you move forward this PR? |
* hpu asr support * add hpu support for tts * enhance device condition * fix lint and coverage * remove env variables * remove cpu_pool --------- Co-authored-by: Haihao Shen <haihao.shen@intel.com> Signed-off-by: Lv, Liang1 <liang1.lv@intel.com>
* Support HPU on ASR/TTS (#290) * hpu asr support * add hpu support for tts * enhance device condition * fix lint and coverage * remove env variables * remove cpu_pool --------- Co-authored-by: Haihao Shen <haihao.shen@intel.com> Signed-off-by: Lv, Liang1 <liang1.lv@intel.com> * Merge finetuning and inference docker into unified one Signed-off-by: Lv, Liang1 <liang1.lv@intel.com> * update docker files Signed-off-by: Lv, Liang1 <liang1.lv@intel.com> * fix path issue Signed-off-by: Lv, Liang1 <liang1.lv@intel.com> --------- Signed-off-by: Lv, Liang1 <liang1.lv@intel.com> Co-authored-by: Spycsh <39623753+Spycsh@users.noreply.github.com> Co-authored-by: Haihao Shen <haihao.shen@intel.com> Co-authored-by: kevinintel <hanwen.chang@intel.com>
Type of Change
hpu hardware enabling
Description
Now ASR/TTS should support HPU/CPU/GPU
Expected Behavior & Potential Risk
Now ASR/TTS should support HPU/CPU/GPU
How has this PR been tested?
Run the two UT on HPU
Dependency Change?
None