-
Notifications
You must be signed in to change notification settings - Fork 219
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
Added mamba and test without set env #1623
base: v1.15-release
Are you sure you want to change the base?
Conversation
Ok, looks good. Validated on all these case.
Perf, good,
No crash anymore, fall back to old way. Perf is half of the 1)
|
When I run these commands with
Any idea why this happens? @zzhang37 This code should be conditional to whether we use Mamba or not. We don't want this piece of code to be executed if another model is run as it will error out since it is likely the kernel has not been downloaded. |
@regisss I saw this issue. We se this when we do in the docker |
What does this PR do?
This is to merge 1.15-release, all issues fixed.
Fixes # (issue)
Before submitting