Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rewrite DRY sampler to be a lot faster #868

Merged
merged 2 commits into from
Dec 4, 2024
Merged

Rewrite DRY sampler to be a lot faster #868

merged 2 commits into from
Dec 4, 2024

Conversation

50h100a
Copy link
Collaborator

@50h100a 50h100a commented Dec 4, 2024

Upgraded from extremely slow to pretty slow.

Possible issues:

  • Calculating the padding around ln 642 seems dumb, surely we have that somewhere?
  • Suspect the ln 681 loop is where the slowness is coming from now, but I'm not sure. Also not sure how to tensorize it atm.

@50h100a 50h100a merged commit 9b56927 into main Dec 4, 2024
5 checks passed
@50h100a 50h100a deleted the dry_zoom branch December 4, 2024 23:50
@Nero10578
Copy link

This is cooking. Thanks for the fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants