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

rwightman/pytorch-image-models integration #70

Merged
merged 8 commits into from
Feb 23, 2021
Merged

rwightman/pytorch-image-models integration #70

merged 8 commits into from
Feb 23, 2021

Conversation

@bfineran
Copy link
Contributor

No description provided.

@bfineran bfineran requested a review from a team February 10, 2021 22:30
@bfineran bfineran self-assigned this Feb 10, 2021
@bfineran bfineran force-pushed the rwightman-integration branch 9 times, most recently from 66ab7cf to d86fb92 Compare February 14, 2021 00:04
markurtz
markurtz suggested changes Feb 17, 2021
View reviewed changes
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.

Left a few comments. Additionally, can we migrate it over to your suggestion of having a folder for integrations? And let's name the root just timm since we're already in the sparseml repo

@markurtz
Copy link
Member

Oh also, can we get the exact checkpoint of the integrated repo this was tested with in the even people have issues with newer versions?

@markurtz markurtz mentioned this pull request Feb 17, 2021
Merged
@bfineran bfineran force-pushed the rwightman-integration branch from d86fb92 to 7974bef Compare February 19, 2021 20:24
bfineran
bfineran commented Feb 19, 2021
View reviewed changes
Copy link
Contributor Author

@bfineran bfineran left a comment

Choose a reason for hiding this comment

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

moved this integrations to a new integrations directory. will move the existing integrations in a follow up PR

@bfineran
Copy link
Contributor Author

Oh also, can we get the exact checkpoint of the integrated repo this was tested with in the even people have issues with newer versions?

I include the commit hash in the doc string

@bfineran bfineran force-pushed the rwightman-integration branch 2 times, most recently from 1b8c593 to 24e05ce Compare February 19, 2021 20:32
markurtz
markurtz suggested changes Feb 19, 2021
View reviewed changes
@bfineran bfineran force-pushed the rwightman-integration branch 2 times, most recently from 1c5a1d4 to 60ab884 Compare February 23, 2021 03:25
markurtz
markurtz suggested changes Feb 23, 2021
View reviewed changes
markurtz
markurtz previously approved these changes Feb 23, 2021
View reviewed changes
@markurtz markurtz requested a review from a team February 23, 2021 15:22
mgoin
mgoin previously approved these changes Feb 23, 2021
View reviewed changes
bfineran
bfineran commented Feb 23, 2021
View reviewed changes
bfineran
bfineran commented Feb 23, 2021
View reviewed changes
bfineran
bfineran commented Feb 23, 2021
View reviewed changes
@bfineran bfineran dismissed stale reviews from mgoin and markurtz via 2ca2ffb February 23, 2021 17:51
@bfineran bfineran merged commit ffcd02e into main Feb 23, 2021
@bfineran bfineran deleted the rwightman-integration branch February 23, 2021 19:28
markurtz added a commit that referenced this pull request Feb 25, 2021
@markurtz @jeanniefinks
@bfineran @eldarkurtic @natuan
Rebase release/0.1 off of main for 0.1.1 (#87)
edcce80 
* GA code, toctree links (#61)

- added tracking for docs output
- added help links for docs output

* Update README.md (#64)

removed placeholder reference to comingsoon repo in favor of active repo

* add decorator for flaky tf sparsity tests (#65)

* enable modifier groups in SparseML recipes (#66)

* enable modifier groups in SparseML recipes
* unit tests for YAML modifier list loading

* make build argument for nightly builds (#63)

* rst url syntax correction (#67)

correcting double slash at the end of URLs with updates to index.rst pre-compilation

* match types explicitly in torch qat quant observer wrappers (#68)

* docs updates (#71)

enhancing left nav for Help; after this merge, the docs need to be rebuilt for this repo so docs.neuralmagic.com can be refreshed. cc @markurtz

* Rename KSModifier to PruningModifier (#76)

* Rename ConstantKSModifier to ConstantPruningModifier

* Rename GradualKSModifier to GMPruningModifier

* Fix broken link for Optimization Recipes (#75)

* Serialize/deserialize MaskedLayer (#69)

* Serialize/deserialize MaskedLayer

* Remove unused symbols

* Register pruning scheduler classes for serialization

* removed ScheduledOptimizer, moved logic to ScheduledModifierManager (#77)

* load recipes directly from SparseZoo (#72)

* load recipes into Managers from sparsezoo stubs
* moving recipe_type handling to SparseZoo only, supporting loading SparseZoo recipe objects

* Revert "removed ScheduledOptimizer, moved logic to ScheduledModifierManager (#77)" (#80)

This reverts commit 6073abb.

* Update for 0.1.1 release (#82)

* Update for 0.1.1 release
- 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 in deepsparse optional install pathway

* missed updating version to 0.1.1

* rename examples directory to integrations (#78)

* rwightman/pytorch-image-models integration (#70)

* load checkpoint file based on sparsezoo recipe in pytorch_vision script (#83)

* ultralytics/yolov5 integration (#73)

* pytorch sparse quantized transfer learning notebook (#81)

* load qat onnx models for conversion from file path (#86)

* Sparsification update (#84)

* Sparsification update
- update sparsification descriptions and move to preferred verbage

* update from comments on deepsparse for sparsification

* 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/index.rst

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

* Update docs/source/index.rst

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

* Update docs/source/index.rst

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

* Update docs/source/index.rst

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

* Update docs/source/recipes.md

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

* fix links in index.rst from reviewed content

* update overviews and taglines from doc

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

* blog style readme for torch sparse-quant TL notebook (#85)

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: Eldar Kurtic <eldar.ciki@gmail.com>
Co-authored-by: Tuan Nguyen <tuan@neuralmagic.com>
markurtz pushed a commit that referenced this pull request Sep 1, 2021
@jeanniefinks
Jfinks badlink archive (#70)
5a651ca 
* Update index.html

stray bad link to clean up for clean archive

* Update index.rst.txt

stray bad link to clean up for clean archive
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

2 more reviewers

@mgoin mgoin mgoin approved these changes

@markurtz markurtz markurtz approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@bfineran bfineran

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@bfineran @markurtz @mgoin