Skip to content
This repository was archived by the owner on Jul 4, 2025. It is now read-only.

Conversation

@vansangpfiev
Copy link
Contributor

Describe Your Changes

Use model.list when run cortex models start cortex chat and cortex run

Fixes Issues

#1300

Self Checklist

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

@vansangpfiev vansangpfiev marked this pull request as ready for review September 23, 2024 07:05
Copy link
Contributor

@dan-menlo dan-menlo left a comment

Choose a reason for hiding this comment

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

Lgtm, thank you @vansangpfiev

Copy link
Contributor

@namchuai namchuai left a comment

Choose a reason for hiding this comment

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

✅ lgtm

@vansangpfiev vansangpfiev merged commit 7c31b71 into feat/new-model-folder Sep 24, 2024
4 checks passed
@vansangpfiev vansangpfiev deleted the feat/new-model-folder_run branch September 24, 2024 04:22
nguyenhoangthuan99 added a commit that referenced this pull request Sep 24, 2024
* feat: pulling interact with new model.list

* feat: return model id when download model success

* feat: model delete for new model.list (#1317)

* feat: models delete for new models data folder structure

* feat: delete model

* feat: cortex chat/run/models start with new model data structure (#1301)

* feat: cortex chat/run/models start with new model data structure

* temp

* fix: use model_id from model service and std::get_line

* f:m

* Model update command/api (#1309)

* Model update command/api

* fix: resume download failed

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

* fix: align github syntax for cuda (#1316)

* fix: require sudo for cortex update (#1318)

* fix: require sudo for cortex update

* fix: comment

* refactor code

* Format code

* Add clean up when finish test

* remove model.list after finish test

* Fix windows CI build

---------

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

* Add more fields to handle when update

---------

Signed-off-by: James <[email protected]>
Co-authored-by: James <[email protected]>
Co-authored-by: vansangpfiev <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants