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

Conversation

horheynm
Copy link
Contributor

@horheynm horheynm commented Feb 28, 2024

Previous commit fails bc the chunks are saved in root model folder.
Tests checks the expected files, where chunks are not included.

Before these commits, chunk folder is deleted each run so it passed. Now if multiple downloading is running, then delete of a folder will have directory to write to

Chunks are now saved in ~/.cache/sparsezoo/neuralmagic/chunks/{stub}/{training_or_deployment_or_folder_name}/{model_data}/chunk1

Combined chunks are saved in
Chunks are now saved in ~/.cache/sparsezoo/neuralmagic/{stub}/training/model.data

bfineran
bfineran previously approved these changes Feb 29, 2024
@bfineran bfineran merged commit b463e83 into main Feb 29, 2024
@bfineran bfineran deleted the download-iso-chunk-folder branch February 29, 2024 16:44
horheynm added a commit that referenced this pull request Feb 29, 2024
* add more info on what chunk is downloading, and make chunk folder foe each file

* fix bug

* comments

* add .cache/sparsezoo/neuralmagic/
bfineran pushed a commit that referenced this pull request Feb 29, 2024
* add more info on what chunk is downloading, and make chunk folder foe each file (#469)

* Download iso chunk folder (#470)

* add more info on what chunk is downloading, and make chunk folder foe each file

* fix bug

* comments

* add .cache/sparsezoo/neuralmagic/

---------

Co-authored-by: Rahul Tuli <rahul@neuralmagic.com>
bfineran pushed a commit that referenced this pull request Mar 8, 2024
* add more info on what chunk is downloading, and make chunk folder foe each file (#469)

* Download iso chunk folder (#470)

* add more info on what chunk is downloading, and make chunk folder foe each file

* fix bug

* comments

* add .cache/sparsezoo/neuralmagic/

* Multiple download bug (#476)

* src/sparsezoo/utils/download.py

* revert readme"
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.

3 participants