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

Support torch.dtype, device, memory_format, layout, etc #144

Closed
Tracked by #139
stellaraccident opened this issue Oct 31, 2023 · 0 comments · Fixed by #162
Closed
Tracked by #139

Support torch.dtype, device, memory_format, layout, etc #144

stellaraccident opened this issue Oct 31, 2023 · 0 comments · Fixed by #162
Milestone

Comments

@stellaraccident
Copy link
Contributor

No description provided.

@stellaraccident stellaraccident added this to the Path to V1 milestone Oct 31, 2023
IanNod pushed a commit that referenced this issue Nov 9, 2023
This PR takes care of
#139.

For the weird tensor meta data, we should only throw error if quantized.
If we start looking at other things like stride, memory_format, etc. the
pytest fails.

For the TODO where we wanted to throw a KeyError, we already check if
parameter.name in node.kwargs in the if statement above, so we won't run
into an invalid key.

Other issues that were part of this importer completeness task have been
addressed and documented.

Fixes #139 
Fixes #140 
Fixes #141 
Fixes #142 
Fixes #143 
Fixes #144
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant