Page MenuHomeFeedback Tracker

[Logging] Add more context info to "Cycle as first waypoint has no sense" rpt warning
Feedback, NormalPublic

Description

Would it be possible please to get more info with this rpt message

ie group name, waypoints, current waypoint, position or so

The message happens way too often to be only a group with just a cycle WP

Yet without more info, its very hard to pinpoint where this coming from

Details

Severity
None
Resolution
Open
Reproducibility
N/A
Operating System
Windows 10 x64
Category
Scripting

Event Timeline

dedmen added a subscriber: dedmen.Jan 16 2024, 4:01 PM

The message already prints the group.
"%s: Cycle as first waypoint has no sense"
is the whole message, and the group gets inserted at the start.

The message is quite bad though.
Cycle finds the nearest waypoint, that has a lower WP index than the cycle itself. It chooses that as next target.

Now if you would for example delete old waypoints that were already executed, cycle would become the "first" waypoint, and it would be messed up.

I can add the WP name and position to the log message
B Alpha 1-1: Cycle as first waypoint has no sense (Cycle WP '' at [4478.810547, 2381.105225, 5.000000])
Like that.
The height is in the middle of that position..

dedmen changed the task status from New to Feedback.Jan 16 2024, 4:05 PM

Good point that scripted WP removal could also lead to that. Atm its quite hard to debug as mention - WaypointCompleted group eventhandler would make this also easier: #T178427