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

Bingo Plugin #7559

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Bingo Plugin #7559

wants to merge 7 commits into from

Conversation

ItsRemer
Copy link

@ItsRemer ItsRemer commented Mar 2, 2025

Enhance your clan’s Bingo events with this plugin—designed to streamline tracking, scoring, and coordination of Bingo challenges within RuneLite. Perfect for hosting engaging, organized Bingo sessions for any clan or community.

@runelite-github-app
Copy link

runelite-github-app bot commented Mar 2, 2025

ItsRemer added 2 commits March 2, 2025 04:49
Added a notification for completion of the full board,
Added a notification for completion of a row,
Added more discord notifications for completions,
More refactoring to the code has been done.
@LlemonDuck
Copy link
Contributor

Don't sleep the shared executor service, it's a shared thread and you will starve other plugins. The screenshot system is already asynchronous and I'd recommend reworking your CompletableFuture to just resolve within requestNextFrameListener or better yet, just react asynchronously as well by queueing resultant handlers after the screenshot is taken.

@LlemonDuck LlemonDuck added the waiting for author waiting for the pr author to make changes or respond to questions label Mar 3, 2025
…hes improving performance in findAndMarkItem and loadSavedItems methods

Optimized Item Processing
Reduced redundant string comparisons and list iterations
Replaced stream operations with direct for-loop for better performance
Eliminated the need to create intermediate collections
Added proper clearing of the lookup map when resetting items
Improved checkForCompletions to only check diagonals when the item is actually on a diagonal
Reduced redundant calculations and boolean variables
Refactored more code
Fully Asynchronous Screenshot System (Replaced blocking CompletableFuture with a callback-based approach
                                      Implemented proper asynchronous handling within the frame listener
                                      Used scheduled tasks for retries instead of thread-blocking sleeps)
Better error handling and validation throughout
Simplified payload creation
Fixed issues with Discord webhook attachments
@runelite-github-app runelite-github-app bot removed the waiting for author waiting for the pr author to make changes or respond to questions label Mar 3, 2025
@ItsRemer ItsRemer marked this pull request as draft March 4, 2025 03:25
@ItsRemer ItsRemer marked this pull request as ready for review March 4, 2025 03:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants