Skip to content

Conversation

@bmyrcha
Copy link
Contributor

@bmyrcha bmyrcha commented Oct 9, 2023

Type of Change

Documentation

Description

Documentation with example of tensor dump for tensor dump using Neural Insights CLI.

Expected Behavior & Potential Risk

User should be able to dump tensor using this instruction.

How has this PR been tested?

Manual execution of commands included in readme.

Dependency Change?

None

@bmyrcha bmyrcha added the documentation Improvements or additions to documentation label Oct 9, 2023
@bmyrcha bmyrcha self-assigned this Oct 9, 2023
@bmyrcha bmyrcha force-pushed the bmyrcha/ni_pytorch_example branch 2 times, most recently from 859a759 to 6658370 Compare October 9, 2023 06:59
Signed-off-by: bmyrcha <bartosz.myrcha@intel.com>
@bmyrcha bmyrcha force-pushed the bmyrcha/ni_pytorch_example branch from 6658370 to 81d3fd0 Compare October 9, 2023 07:02
@chensuyue
Copy link
Contributor

Signed-off-by: bmyrcha <bartosz.myrcha@intel.com>
@bmyrcha bmyrcha force-pushed the bmyrcha/ni_pytorch_example branch from 51227fc to d02a358 Compare October 9, 2023 07:33
@bmyrcha
Copy link
Contributor Author

bmyrcha commented Oct 9, 2023

Copy link
Contributor

@hshen14 hshen14 left a comment

Choose a reason for hiding this comment

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

LGTM

@hshen14 hshen14 merged commit 43af08b into master Oct 9, 2023
@hshen14 hshen14 deleted the bmyrcha/ni_pytorch_example branch October 9, 2023 07:58
bmyrcha added a commit that referenced this pull request Oct 24, 2023
* Add example of PyTorch model tensor dump with Neural Insights

Signed-off-by: bmyrcha <bartosz.myrcha@intel.com>

* Move instruction to correct directory

Signed-off-by: bmyrcha <bartosz.myrcha@intel.com>

---------

Signed-off-by: bmyrcha <bartosz.myrcha@intel.com>
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

Successfully merging this pull request may close these issues.

5 participants