Skip to content

Timeless Jewel Finder: Intelligent batching - #8620

Closed
Kojoley wants to merge 1 commit into
PathOfBuildingCommunity:devfrom
Kojoley:timeless-jewel-finder-intelligent-batching
Closed

Timeless Jewel Finder: Intelligent batching#8620
Kojoley wants to merge 1 commit into
PathOfBuildingCommunity:devfrom
Kojoley:timeless-jewel-finder-intelligent-batching

Conversation

@Kojoley

@KojoleyKojoley commented Jun 6, 2025

Copy link
Copy Markdown
Contributor

Instead of fixed batch size of 4 it's now automatically sized to include similar power jewels.

Example of how it works now (POB https://pobb.in/N7P7HjJ1_KN2)
image
image
image
image

Instead of fixed batch size of 4 it's now automatically sized to include similar power jewels.
@Kojoley
Kojoleyforce-pushed the timeless-jewel-finder-intelligent-batching branch from ebfe128 to 35d9ff6CompareJune 13, 2025 12:06
@Kojoley

Copy link
Copy Markdown
ContributorAuthor

The lint error is in a file I not even touched, it seems to be fixed in #8622, I've rebased the PR.

@LocalIdentity

Copy link
Copy Markdown
Contributor

Superseded by #9805

@Kojoley

Kojoley commented May 12, 2026

Copy link
Copy Markdown
ContributorAuthor

Superseded by #9805

It does not, it's a worse version of my patch. Not batching by similar power just increased batch limit.

Sign up for freeto 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

@Kojoley@LocalIdentity