这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@eustas
Copy link
Contributor

@eustas eustas commented Aug 11, 2025

3.5-4x faster upsampling

@eustas eustas force-pushed the faups branch 2 times, most recently from 8507593 to 2497230 Compare August 11, 2025 12:15
@eustas eustas added the CI:full Label to attach to a PR to run the full CI workflow and not just the regular PR workflows label Aug 11, 2025
@eustas eustas requested a review from szabadka August 11, 2025 12:39
@eustas eustas mentioned this pull request Aug 9, 2025
9 tasks
@jonnyawsom3
Copy link
Collaborator

Good work. Should make --resampling properly scale decode speed with internal resolution.

@jonnyawsom3
Copy link
Collaborator

Does this only apply to 8x upsampling? Would it be difficult to do 4x and 2x too? 8x is used for the LF but 2x is used at or above distance 10.

@eustas
Copy link
Contributor Author

eustas commented Aug 18, 2025

It affects all 3 resampings (that is why speedup us a range, not a single number); though, I've tested it only on AVX2; most likely other vector engines will benefit. Not unlikely scalar too.

@eustas eustas requested a review from mo271 August 18, 2025 08:24
@eustas eustas added this pull request to the merge queue Aug 18, 2025
Merged via the queue into libjxl:main with commit 4717d7d Aug 18, 2025
89 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI:full Label to attach to a PR to run the full CI workflow and not just the regular PR workflows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants