Skip to content

afollow fallback, settings persistence#1

Open
cjmanca wants to merge 3 commits into
aquietone:mainfrom
cjmanca:main
Open

afollow fallback, settings persistence#1
cjmanca wants to merge 3 commits into
aquietone:mainfrom
cjmanca:main

Conversation

@cjmanca
Copy link
Copy Markdown

@cjmanca cjmanca commented May 21, 2025

  • Add /afollow as a fallback for when no nav path can be found.
  • Option for minimum distance to activate /afollow (so it doesn't try to do it from across the zone)
  • Now saves most settings to a config file
  • Added several options to the Chase TLO which should respond similarly to AdvPath's TLO, so you can just find/replace AdvPath/afollow to Chase in existing macros.

- Add /afollow as a fallback for when no nav path can be found.
- Option for minimum distance to activate /afollow (so it doesn't try to do it from across the zone)
- Now saves most settings to a config file
- Added several options to the Chase TLO which should respond similarly to AdvPath's TLO, so you can just find/replace AdvPath/afollow to Chase in existing macros.
Comment thread chase.lua Outdated
Comment thread chase.lua Outdated
Comment thread chase.lua Outdated
Comment thread chase.lua
end

local tlomembers = {
Active = function() return 'bool', true end,
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should this true be not PAUSED?

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

all these extra members that amount to nearly identical information seems like a lot of bloat just to match afollow

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

According to the afollow information, "Active" always returns true as long as the plugin is loaded, hence why it's just "true" here.

Comment thread chase.lua
local PREFIX = '\aw[\agCHASE\aw] \ay'
local Settings = { CHASE = '', ROLE = 'none', DISTANCE = 30, STOP_DISTANCE = 10, open_gui = true, MAX_AFOLLOW_DISTANCE = 100 }

local RUNNING = false
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the differentiation between running and paused seems odd to me, does it add much to not just have 1 value?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree, I mainly added it for feature parity with afollow. As I was adding/using it, it seemed pointless to have both myself, and I'm not sure why afollow did that either.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My only thought would be that it was intended to make it easier on macro writers, since pause is usually intended to be temporary. So pause to cast a spell, and after just unpause if paused with no extra logic needed. But they're essentially the same thing.

Comment thread chase.lua Outdated
Comment thread chase.lua Outdated
Comment thread chase.lua
Comment thread chase.lua
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