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

Improve documentation when exporting a recipe #83

Closed
pacospace opened this issue Sep 15, 2021 · 2 comments
Closed

Improve documentation when exporting a recipe #83

pacospace opened this issue Sep 15, 2021 · 2 comments
Labels
documentation Improvements or additions to documentation

Comments

@pacospace
Copy link

What is the URL, file, or UI containing proposed doc change
Where does one find the original content or where would this change go?

This change would go in the main README of sparsify repository.

What is the current content or situation in question
https://github.com/neuralmagic/sparsify#exporting-a-recipe

What is the proposed change
In the main README it should be mentioned that tensorflow or torch should be installed with sparsify to be able to export them.
Screenshot from 2021-09-15 17-52-53

Moreover it should be mentioned the range of version for torch or tensorflow that can be handled:

Screenshot from 2021-09-15 17-53-41

Additional context
Related-To new tutorial with NerualMagic: AICoE/elyra-aidevsecops-tutorial#297

@markurtz
Copy link
Member

Thanks for this suggestion @pacospace, we will get this updated!

@markurtz
Copy link
Member

Closing this out now as we added in a note in the README. We'll work towards easier flows with this as well through optional pathways in pip installs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

2 participants