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 Skyguard/Ogri'la Flights #25

Open
beren12 opened this issue Jan 31, 2022 · 1 comment
Open

Add Skyguard/Ogri'la Flights #25

beren12 opened this issue Jan 31, 2022 · 1 comment
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@beren12
Copy link
Collaborator

beren12 commented Jan 31, 2022

They use a dialog like the druid FPs but don't give a perfect location for the destination. Could you possibly assist @illiath? I can screenshot the dialogs if needed.

@illiath
Copy link
Owner

illiath commented Feb 1, 2022

I can have a look. Screenshots would be useful.

So, quickly rechecking my code, check out Timer.lua, line 166, that starts the "Druid" flight path capture stuff. Originally, this code had hard-coded "target" locations because the druid start positions moved a little on the map, but I was able to decrease the resolution of the map x/y coordinates sufficiently not to impede the locations for any other points so that was removed. You might need to therefore add a new entry style that has them force-set.

That said, screenshots of the EFM display to show where it thinks the nodes are would be great.
Also what would be great is a dump of the node information for the node that "wanders". It might be possible to adjust for this, depending on the situation.

However, if the nodes move considerably, then this could not be adjusted for in the node settings and so each target point would need a new node definition, and so figuring out which node the user is able to fly to right now, and showing that information would require an additional abstration layer.

@beren12 beren12 added enhancement New feature or request help wanted Extra attention is needed labels Aug 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants