TensorRT icon indicating copy to clipboard operation
TensorRT copied to clipboard

feat: keep model's original input signature

Open bowang007 opened this issue 3 years ago • 4 comments

Signed-off-by: Bo Wang [email protected]

Description

There are some requests that users doesn't want to change the models' inputs signature for input, which could make it difficult to use in a deployment environment. We can keep original inputs signature.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Checklist:

  • [ ] My code follows the style guidelines of this project (You can use the linters)
  • [ ] I have performed a self-review of my own code
  • [ ] I have commented my code, particularly in hard-to-understand areas and hacks
  • [ ] I have made corresponding changes to the documentation
  • [ ] I have added tests to verify my fix or my feature
  • [ ] New and existing unit tests pass locally with my changes
  • [ ] I have added the relevant labels to my PR in so that relevant reviewers are notified

bowang007 avatar Aug 04 '22 07:08 bowang007

Does this solution not change the input names anyway?

narendasan avatar Aug 04 '22 13:08 narendasan

Does this solution not change the input names anyway?

yes, that's what we expect. Any thought on this? curious about whether it's really necessary or not.

bowang007 avatar Aug 04 '22 22:08 bowang007

@bowang007 how does this effect runtime? does this change our convention for providing inputs? I don't think it should but just to make sure

narendasan avatar Aug 15 '22 20:08 narendasan

@bowang007 how does this effect runtime? does this change our convention for providing inputs? I don't think it should but just to make sure

I don't think so.

bowang007 avatar Aug 15 '22 23:08 bowang007

This PR has not seen activity for 90 days, Remove stale label or comment or this will be closed in 10 days

github-actions[bot] avatar Nov 14 '22 00:11 github-actions[bot]