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

Add Tree Run Helper #1586

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

Add Tree Run Helper #1586

wants to merge 20 commits into from

Conversation

Zoinkwiz
Copy link
Owner

@Zoinkwiz Zoinkwiz commented Jun 6, 2024

This is a helper to hopefully provide some nice utility such as the auto-bank for tree runs, as well as help keep make sure you don't accidentally miss out on paying for protection. There's various toggles for preference.

I think it could use further guidance, and really could benefit from some way to move about the order depending on preference. I think that needs to be a project in its own right though.

Kerpackie and others added 20 commits June 5, 2024 11:28
Starts the tree run with some conditions
Adds a FarmingUtils class which handles configuration options for the farm helpers.
Add createSeedRequirement which providing an easy way to set up seed/sapling requirements without repeating code.
Adds configuration options
Starts the patch steps
Adds a FarmConfigChangeHandler which (better?) handles the onChange events for the Farming Helpers.
It actually works now, and is now actually shorter than originally.
Adds Hardwood Steps to TreeHelper
Adds base requirements and steps
Adds steps so that they function correctly.
Fixes all coords so that highlight will function correctly.
Check using Varbit the current state of the trees nearby, allowing for better differentiation of trees in a Checked/Unchecked and stump state to accommodate clearing.
@Kerpackie
Copy link
Contributor

Thanks for taking the time out to finish this for me, great to see it finally completed!

@pajlada
Copy link
Contributor

pajlada commented Jun 23, 2024

Does not recommend a rake, although maybe that's correct since users should know about raking patches by this point and would just clutter the guide up

The Gnome Stronghold step skipped the normal tree, and only did the fruit tree for me

The dialog steps for chopping a tree down is missing the slow dialog steps, they should be
step.addDialogSteps("Would you chop my tree down for me?","I can't be bothered - I'd rather pay you to do it.", "Here's 200 Coins - chop my tree down please.", "Yes.");

Would be nice if Elkoy was highlighted to guide you out (i know the guide tells you)

When I did the fossil island hard trees, it told me I was done after just planting one of the 3 trees

@Kerpackie
Copy link
Contributor

When I did the fossil island hard trees, it told me I was done after just planting one of the 3 trees

This could be an issue with how the varb for the trees updates. There is a shared varb for a bunch of patches, and it gets updated based on how close you are to a specific patch. Considering how close the fossil island plots are, if they are all sharing the same varb then the zoning will need to be really specific for it or it would skip the step.

@Brandon1212b
Copy link

From this thread it sounds like there are tree runs in quest helper but I am not seeing it currently. Is there any way I can add this?Im not too familiar with Github, I am sorry if this is a noob question.

@kcjg94
Copy link
Contributor

kcjg94 commented Jul 27, 2024

From this thread it sounds like there are tree runs in quest helper but I am not seeing it currently. Is there any way I can add this?Im not too familiar with Github, I am sorry if this is a noob question.

It is currently in development. The tree run helper is not on the version of Quest Helper available in the plugin hub through Runelite.

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.

5 participants