Skip to content

Multiple Inputs Support#1431

Merged
yarden-yagil-sony merged 19 commits intomainfrom
multiple_inputs_support
May 7, 2025
Merged

Multiple Inputs Support#1431
yarden-yagil-sony merged 19 commits intomainfrom
multiple_inputs_support

Conversation

@yarden-yagil-sony
Copy link
Copy Markdown
Contributor

@yarden-yagil-sony yarden-yagil-sony commented Apr 29, 2025

Pull Request Description:

Support multiple inputs in onnx export

Checklist before requesting a review:

  • I set the appropriate labels on the pull request.
  • I have added/updated the release note draft (if necessary).
  • I have updated the documentation to reflect my changes (if necessary).
  • All function and files are well documented.
  • All function and classes have type hints.
  • There is a licenses in all file.
  • The function and variable names are informative.
  • I have added new unittest (if necessary).
  • I have checked for code duplications.

Copy link
Copy Markdown
Contributor

@Idan-BenAmi Idan-BenAmi left a comment

Choose a reason for hiding this comment

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

Better to add a test for this

@yarden-yagil-sony yarden-yagil-sony merged commit 51d2aaf into main May 7, 2025
31 checks passed
@ofirgo ofirgo deleted the multiple_inputs_support branch July 2, 2025 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants