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

Conversation

@bnellnm
Copy link
Member

@bnellnm bnellnm commented Feb 23, 2021

This adds support for batch-splitting in the engine, i.e. if the number of the sockets on the system is > 1, then the model is split into batch_size/num_sockets smaller models which each run on their own socket.

Batch splitting will be enabled whenever num_sockets >1.

bnellnm and others added 3 commits February 21, 2021 01:05
- update python version to 0.1.1
- setup.py add in version parts and _VERSION_MAJOR_MINOR for more flexibility with dependencies between neural magic packages
@bnellnm bnellnm requested review from markurtz and mgoin February 23, 2021 19:02
@bnellnm bnellnm self-assigned this Feb 23, 2021
mgoin
mgoin previously approved these changes Feb 23, 2021
Copy link
Member

@mgoin mgoin left a comment

Choose a reason for hiding this comment

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

LGTM, thanks Bill

Copy link
Member

@markurtz markurtz left a comment

Choose a reason for hiding this comment

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

Need to run make style to format it, other than that looks good

@bnellnm bnellnm merged commit ae5a8c7 into main Feb 25, 2021
@bnellnm bnellnm deleted the batch-splitting branch February 26, 2021 02:00
markurtz added a commit that referenced this pull request Feb 26, 2021
* Update icon name (#41)

* Update index.rst (#43)

adding license chip for nme; substituting in sparseml for deepsparse value in shields.io until we can debug why deepsparse won't work.

* Update README.md (#44)

removed comingsoon referene and adding our active repo for shields.io badge

* Makefile build argument for nightly builds (#45)

* docs updates (#46)

- correcting double-slash URL issue
- enhancing left nav for Help
- misc content updates

* Update for 0.1.1 release (#49)

- update python version to 0.1.1
- setup.py add in version parts and _VERSION_MAJOR_MINOR for more flexibility with dependencies between neural magic packages

* add compile engine from SparseZoo stub and benchmarking progress bar (#48)

* Sparsification update (#51)

* Sparsification update
- update sparsification descriptions and move to preferred verbage
- update classification examples to resnet50

* update from comments

* Update README.md

Co-authored-by: Jeannie Finks <74554921+jeanniefinks@users.noreply.github.com>

* Update README.md

Co-authored-by: Jeannie Finks <74554921+jeanniefinks@users.noreply.github.com>

* Update README.md

Co-authored-by: Jeannie Finks <74554921+jeanniefinks@users.noreply.github.com>

* Update README.md

Co-authored-by: Jeannie Finks <74554921+jeanniefinks@users.noreply.github.com>

* Update README.md

Co-authored-by: Jeannie Finks <74554921+jeanniefinks@users.noreply.github.com>

* Update README.md

Co-authored-by: Jeannie Finks <74554921+jeanniefinks@users.noreply.github.com>

* Update docs/source/quicktour.md

Co-authored-by: Jeannie Finks <74554921+jeanniefinks@users.noreply.github.com>

* update for changes found in sparsezoo

* fix links in index.rst for reviewd content

* update component overview and tagline from doc

* update from comments

Co-authored-by: Jeannie Finks <74554921+jeanniefinks@users.noreply.github.com>

* Add resnet50 benchmark script (#52)

* Add resnet50 benchmark script

* Add Ben's warning and ISA print

* Remove old file

* Batch splitting (#50)

* Add support for batch splitting

* Update for 0.1.1 release (#49)

- update python version to 0.1.1
- setup.py add in version parts and _VERSION_MAJOR_MINOR for more flexibility with dependencies between neural magic packages

* Add support for batch splitting

* Remove use_batch_splitting parameter

* Run style on changes

Co-authored-by: Mark Kurtz <mark@neuralmagic.com>

* Rm: blog link, 7x (#54)

* Update README.md and index.rst

Temporarily removing references to 7x and blog. Will return after it's live.

Co-authored-by: Michael Goin <michael@neuralmagic.com>
Co-authored-by: Jeannie Finks (NM) <74554921+jeanniefinks@users.noreply.github.com>
Co-authored-by: Benjamin Fineran <bfineran@users.noreply.github.com>
Co-authored-by: bnellnm <49004751+bnellnm@users.noreply.github.com>
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