Skip to content

Conversation

@yiliu30
Copy link
Contributor

@yiliu30 yiliu30 commented Dec 5, 2022

Signed-off-by: yiliu30 [email protected]

Type of Change

  • Others: Optimize user experience
  • API changed or not: None

Description

Try to import hyperopt and sigopt from the local environment when user specify the strategy as tpe or sigopt

How has this PR been tested?

Pre-CI

Dependency Change?

Remove hyperopt and sigopt from the test requirements list.

@chensuyue chensuyue added the enhancement New feature or request label Dec 5, 2022
@yiliu30 yiliu30 requested review from ftian1 and hshen14 December 5, 2022 09:01
@chensuyue chensuyue merged commit 4d16046 into master Dec 5, 2022
@chensuyue chensuyue deleted the ly/optional_dependency branch December 5, 2022 12:57
PenghuiCheng pushed a commit that referenced this pull request Dec 6, 2022
zehao-intel pushed a commit that referenced this pull request Dec 9, 2022
zehao-intel pushed a commit that referenced this pull request Dec 20, 2022
VincyZhang pushed a commit that referenced this pull request Feb 12, 2023
* ttodo: fix u8 store code.

* todo:shuffle 4 xmm into 1 zmm when apply quantize fusion.

* reduce lnorm param num,tb benchmark

* adapt benchmark report ci

Co-authored-by: Ding, Yi1 <[email protected]>
yiliu30 added a commit that referenced this pull request Apr 30, 2025
 (#213)

[SW-224851]Fix `PatchedMoeMatmul` and Get `num_experts` from Module 
---------

Signed-off-by: Yi Liu <[email protected]>
Co-authored-by: Yi Liu <[email protected]>
Co-authored-by: Linoy Buchnik <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants