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

fix: add compatible check #2342

Merged
merged 10 commits into from
Mar 14, 2024
Merged

Conversation

namchuai
Copy link
Collaborator

Describe Your Changes

Fixes Issues

  • Closes #
  • Closes #

Self Checklist

  • Added relevant comments, esp in complex areas
  • Updated docs (for bug fixes / features)
  • Created issues for follow-up changes or refactoring needed

@namchuai namchuai requested a review from louis-menlo March 14, 2024 02:48
@github-actions github-actions bot added the type: bug Something isn't working label Mar 14, 2024
@namchuai namchuai force-pushed the fix/add-compatible-check branch from bbfb6d7 to ef05857 Compare March 14, 2024 03:01
@namchuai namchuai force-pushed the fix/add-compatible-check branch from 46b06cc to 0f11ce2 Compare March 14, 2024 03:15
@louis-menlo louis-menlo force-pushed the fix/add-compatible-check branch from f461954 to 2905c51 Compare March 14, 2024 04:13
@louis-menlo louis-menlo merged commit e94f603 into tensorrt-llm-extension Mar 14, 2024
@louis-menlo louis-menlo deleted the fix/add-compatible-check branch March 14, 2024 04:49
louis-menlo added a commit that referenced this pull request Mar 14, 2024
* feat: tensorrt-llm-extension

* fix: loading

* feat: add download tensorrt llm runner

Signed-off-by: James <[email protected]>

* feat: update to rollupjs instead of webpack for monitoring extension

Signed-off-by: James <[email protected]>

* feat: move update nvidia info to monitor extension

Signed-off-by: James <[email protected]>

* allow download tensorrt

Signed-off-by: James <[email protected]>

* update

Signed-off-by: James <[email protected]>

* allow download tensor rt based on gpu setting

Signed-off-by: James <[email protected]>

* update downloaded models

Signed-off-by: James <[email protected]>

* feat: add extension compatibility

* dynamic tensor rt engines

Signed-off-by: James <[email protected]>

* update models

Signed-off-by: James <[email protected]>

* chore: remove ts-ignore

* feat: getting installation state from extension

Signed-off-by: James <[email protected]>

* chore: adding type for decompress

Signed-off-by: James <[email protected]>

* feat: update according Louis's comment

Signed-off-by: James <[email protected]>

* feat: add progress for installing extension

Signed-off-by: James <[email protected]>

* chore: remove args from extension installation

* fix: model download does not work properly

* fix: do not allow user to stop tensorrtllm inference

* fix: extension installed style

* fix: download tensorrt does not update state

Signed-off-by: James <[email protected]>

* chore: replace int4 by fl16

* feat: modal for installing extension

Signed-off-by: James <[email protected]>

* fix: start download immediately after press install

Signed-off-by: James <[email protected]>

* fix: error switching between engines

* feat: rename inference provider to ai engine and refactor to core

* fix: missing ulid

* fix: core bundler

* feat: add cancel extension installing

Signed-off-by: James <[email protected]>

* remove mocking for mac

Signed-off-by: James <[email protected]>

* fix: show models only when extension is ready

* add tensorrt badge for model

Signed-off-by: James <[email protected]>

* fix: copy

* fix: add compatible check (#2342)

* fix: add compatible check

Signed-off-by: James <[email protected]>

* fix: copy

* fix: font

* fix: copy

* fix: broken monitoring extension

* chore: bump engine

* fix: copy

* fix: model copy

* fix: copy

* fix: model json

---------

Signed-off-by: James <[email protected]>
Co-authored-by: James <[email protected]>
Co-authored-by: Louis <[email protected]>

* fix: vulkan support

* fix: installation button padding

* fix: empty script

* fix: remove hard code string

---------

Signed-off-by: James <[email protected]>
Co-authored-by: James <[email protected]>
Co-authored-by: NamH <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Something isn't working
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants