Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Arc Stable version test #10087

Merged
merged 8 commits into from
Feb 6, 2024
Merged

Conversation

Ricky-Ting
Copy link
Contributor

Description

1. Why the change?

  1. Add batch_size to arc stable version release test. https://github.com/analytics-zoo/nano/issues/1023
  2. Add a new category of excludes in run.py: model_id:input_len:batch_size.
  3. Modify oneapi environments.

Copy link
Contributor

@liu-shaojun liu-shaojun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@@ -956,7 +956,8 @@ def run_transformer_autocast_bf16( repo_id,
if excludes:
for in_out in conf['in_out_pairs']:
model_id_input = model + ':' + in_out.split('-')[0]
if model_id_input in excludes:
model_id_input_batch_size = model + ':' + in_out.split('-')[0] + ':' + str(conf['batch_size'])
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

model_id_input_batch_size = model_id_input + + ':' + str(conf['batch_size']) ?

@hkvision hkvision merged commit 7f531cd into intel-analytics:main Feb 6, 2024
15 checks passed
Jasonzzt pushed a commit to Jasonzzt/BigDL that referenced this pull request Feb 19, 2024
* add batch_size in stable version test

* add batch_size in excludes

* add excludes for batch_size

* fix ci

* triger regression test

* fix xpu version

* disable ci

* address kai's comment

---------

Co-authored-by: Ariadne <[email protected]>
liu-shaojun pushed a commit that referenced this pull request Mar 25, 2024
* add batch_size in stable version test

* add batch_size in excludes

* add excludes for batch_size

* fix ci

* triger regression test

* fix xpu version

* disable ci

* address kai's comment

---------

Co-authored-by: Ariadne <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants