Skip to content

Conversation

@3l1
Copy link
Contributor

@3l1 3l1 commented Dec 25, 2025

Summary:
Dual LUT Strategy (from Bolt reference)

  1. Range LUT: Covers full int16 range, LOW resolution at small values
  2. Resolution LUT: Scaled input (x*128), HIGH resolution for codes 0-256

Interpolation Weights

  • x <= 255: Use resolution_lut (high accuracy for small values)
  • x >= 256: Use range_lut (works fine for large values)
  • Transition at 255/256 to avoid interpolation artifacts

Next Steps

  • Full test suite validation
  • Performance benchmarking
  • Reduce number of rescale ops

Reviewed By: digantdesai

Differential Revision: D87120925

Summary:
Dual LUT Strategy (from Bolt reference)
1. **Range LUT**: Covers full int16 range, LOW resolution at small values
2. **Resolution LUT**: Scaled input (x*128), HIGH resolution for codes 0-256

 Interpolation Weights
- **x <= 255**: Use resolution_lut (high accuracy for small values)
- **x >= 256**: Use range_lut (works fine for large values)
- **Transition at 255/256** to avoid interpolation artifacts

Next Steps

- Full test suite validation
- Performance benchmarking
- Reduce number of rescale ops

Reviewed By: digantdesai

Differential Revision: D87120925
@3l1 3l1 requested a review from digantdesai as a code owner December 25, 2025 18:50
@pytorch-bot
Copy link

pytorch-bot bot commented Dec 25, 2025

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/16396

Note: Links to docs will display an error until the docs builds have been completed.

❌ 4 New Failures, 1 Unrelated Failure

As of commit 5060a1c with merge base 5a64056 (image):

NEW FAILURES - The following jobs have failed:

UNSTABLE - The following job is marked as unstable, possibly due to flakiness on trunk:

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Dec 25, 2025
@meta-codesync
Copy link

meta-codesync bot commented Dec 25, 2025

@3l1 has exported this pull request. If you are a Meta employee, you can view the originating Diff in D87120925.

Copy link
Contributor

@digantdesai digantdesai left a comment

Choose a reason for hiding this comment

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

Review automatically exported from Phabricator review in Meta.

@3l1
Copy link
Contributor Author

3l1 commented Dec 25, 2025

@pytorchbot label "release notes: none"

@pytorch-bot pytorch-bot bot added the release notes: none Do not include this in the release notes label Dec 25, 2025
@3l1
Copy link
Contributor Author

3l1 commented Dec 25, 2025

NOTE: I have pending changes before merging !!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported meta-exported release notes: none Do not include this in the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants