[Feature]: AutoDeploy: discussion where/how/when to enforce strict export IR
Open
Nobody has claimed this yet.
AutoDeploy
feature request
- Dominant language
- Python
- Stars
- 14.7k
- Forks
- 2.8k
- Avg merge
- 2d 23h
- Merged PRs (30d)
- 489
Description
🚀 The feature, motivation and pitch
Right now, we technically impose a strict requirement to keep our fx graph consistent with the export IR specifications. However, we already break this assumption in multiple places and hence should revisit this assumption
- Quasi-dynamic shapes in prepare_metadata and using batch_info tensor to split sequences
- In-place update with cuda_cached_causal_conv
- Irregular function for gather_logits with undeclared dynamic shape
Let's revisit the assumption of enforcing a strict export IR during/after transforms!
Alternatives
No response
Additional context
No response
Before submitting a new issue...
- Make sure you already searched for relevant issues, and checked the documentation and examples for answers to frequently asked questions.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.