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

feat: model delete for new model.list #1317

Merged

Conversation

vansangpfiev
Copy link
Contributor

@vansangpfiev vansangpfiev commented Sep 24, 2024

Describe Your Changes

  • For imported model: only delete yml file
  • For others: delete all yml, gguf, ...

Fixes Issues

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 changed the title Feat/new model folder model delete feat: model delete for new model.list Sep 24, 2024
@vansangpfiev vansangpfiev marked this pull request as ready for review September 24, 2024 02:25
@vansangpfiev vansangpfiev merged commit e89a72f into feat/new-model-folder Sep 24, 2024
4 checks passed
@vansangpfiev vansangpfiev deleted the feat/new-model-folder_model-delete branch September 24, 2024 03:45
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 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.

2 participants