-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
[JAX]: Support jax.lax.select_n operation for JAX #28025
Conversation
Signed-off-by: 11happy <[email protected]>
build_jenkins |
Signed-off-by: 11happy <[email protected]>
humble ping! @rkazants |
Hello @rkazants, please let me know if there's anything needed from my end. |
Signed-off-by: 11happy <[email protected]>
build_jenkins |
Signed-off-by: 11happy <[email protected]>
Signed-off-by: 11happy <[email protected]>
build_jenkins |
@11happy, please pay attention to failing CI jobs (code-style, build, etc.), it needs to be fixed. |
|
||
@pytest.mark.parametrize("input_shape", [[],[1],[2,3],[4,5,6],[7,8,9,10]]) | ||
@pytest.mark.parametrize("input_type", [np.int32, np.int64, bool]) | ||
@pytest.mark.parametrize("case_num", [1,2,3,4,5,6,7,8,9,10]) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let use case_num starting from 2. Value 1 is impractical.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
build_jenkins |
Let CI complete and check failing jobs in GHA. You may see build issues that need to be fixed. |
Signed-off-by: 11happy <[email protected]>
build_jenkins |
@11happy, looks good. Let us see CI results. If it is fine, I will merge it. Best regards, |
@11happy, please fix CI errors. Check GHA jobs. Example:
|
Signed-off-by: 11happy <[email protected]>
I have removed these extra semi colons. |
Signed-off-by: 11happy <[email protected]>
Signed-off-by: 11happy <[email protected]>
build_jenkins |
I saw this gha fail:
How can I resolve this? |
build_jenkins |
It looks a problem of JAX utils itself that helps to convert JAX model to TF model. I switched off a path via TF FE that is deprecated anyway. Best regards, |
build_jenkins |
1 similar comment
build_jenkins |
Overview:
This pull request fixes #26570.
Testing:
Dependencies:
CC: