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

Fix shape and reformat free tensor handling in the input byte size check #97

Merged
merged 7 commits into from
Jul 27, 2024

Conversation

pskiran1
Copy link
Member

@pskiran1 pskiran1 commented Jul 14, 2024

  • Added support for the is_non_linear_format_io flag in the model configuration to identify whether input/output tensors use a non-linear IO format.
  • Modified the auto-complete configuration to correctly initialize the is_shape_tensor and is_non_linear_format_io flags.
  • For tensors using the non-linear IO format (reformat-free tensors), the backend now validates the input byte size.

tanmayv25
tanmayv25 previously approved these changes Jul 18, 2024
GuanLuo
GuanLuo previously approved these changes Jul 18, 2024
@pskiran1 pskiran1 dismissed stale reviews from GuanLuo and tanmayv25 via 5b8c263 July 19, 2024 08:59
@pskiran1
Copy link
Member Author

@tanmayv25, Thank you for the approval. I added a new commit 5b8c263 to improve the error message to match the remaining tensor types error message in the core library. Could you please re-approve? Thank you.

@pskiran1 pskiran1 requested a review from tanmayv25 July 22, 2024 14:38
@pskiran1 pskiran1 merged commit 8fe8277 into main Jul 27, 2024
1 check passed
@pskiran1 pskiran1 deleted the spolisetty_dlis_6805 branch July 30, 2024 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants