Skip to content

fix(dynamo): harden dynamic converter boundaries - #4632

Open
narendasan wants to merge 1 commit into
narendasan/fix-dynamic-scatter-index-putfrom
narendasan/fix-converter-profile-edge-cases
Open

fix(dynamo): harden dynamic converter boundaries#4632
narendasan wants to merge 1 commit into
narendasan/fix-dynamic-scatter-index-putfrom
narendasan/fix-converter-profile-edge-cases

Conversation

@narendasan

Copy link
Copy Markdown
Collaborator

Summary

  • clamp finite slice stops against a dynamic runtime axis extent
  • keep rank-8-and-higher tensors out of TensorRT partitions at both producer and boundary checks
  • compute optimization-profile opt extents as the true midpoint between min and max
  • handle unbounded symbolic range limits without calling int() on infinity-like sentinels

Issues

Closes #4607
Closes #4608
Closes #4610
Closes #4611

Tests

Stack

6 of 7. Base: #4631. Follow-up: narendasan/fix-symbolic-converter-parameters.

@github-actions github-actions Bot added component: tests Issues re: Tests component: conversion Issues re: Conversion stage component: core Issues re: The core compiler component: converters Issues re: Specific op converters component: api [Python] Issues re: Python API component: dynamo Issues relating to the `torch.compile` or `torch._dynamo.export` paths labels Aug 28, 2026
@meta-cla meta-cla Bot added the cla signed label Aug 28, 2026
@github-actions
github-actions Bot requested a review from zewenli98 August 28, 2026 03:02
@narendasan
narendasan requested a review from apbose August 28, 2026 03:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla signed component: api [Python] Issues re: Python API component: conversion Issues re: Conversion stage component: converters Issues re: Specific op converters component: core Issues re: The core compiler component: dynamo Issues relating to the `torch.compile` or `torch._dynamo.export` paths component: tests Issues re: Tests

Projects

None yet

1 participant