Project

Profile

Help

HostedRedmine.com has moved to the Planio platform. All logins and passwords remained the same. All users will be able to login and use Redmine just as before. Read more...

Bug #859105

Sandbox (fueled) caravels get stuck while they still have MPs (master, various clients)

Added by Chippo Elder over 2 years ago. Updated 3 days ago.

Status:
New
Priority:
Normal
Assignee:
-
Category:
General
Sprint/Milestone:
Start date:
Due date:
% Done:

0%

Estimated time:

Description

If you load the attached savegame (3.0.92-dev (modified 486bce2328)), and press Turn Done, you'll have a new caravel in Delhi [l tgt="city" id=294 name="Delhi" /]. Select that caravel and move it. It gets stuck after moving just one tile. Move the other caravel a few tiles and it also gets stuck.

This happens in gtk and qt clients.

If you load the previous turn's savegame (also attached), and press Turn Done twice and follow the same procedure, the same thing happens.

freeciv-T0113-Y00240-auto.sav.xz (139 KB) freeciv-T0113-Y00240-auto.sav.xz Chippo Elder, 2020-02-07 01:34 PM
freeciv-T0112-Y00220-auto.sav.xz (138 KB) freeciv-T0112-Y00220-auto.sav.xz Chippo Elder, 2020-02-07 01:46 PM

History

#1 Updated by Jacob Nevins over 2 years ago

  • Subject changed from Caravels get stuck while they still have MPs (master, various clients) to Sandbox (fueled) caravels get stuck while they still have MPs (master, various clients)
  • Sprint/Milestone set to 3.0.0

This is with sandbox ruleset. In my copy, Caravels are fueled units which refuel at coasts ("Unit has to be next to safe coast, in a city or a base after 3 turns") using the new "Coast" semantics added in gna patch #6979 (debuting in 3.0, but I haven't checked if S3_0 has the symptom too).

I can reproduce the "gets stuck" behaviour wrt using (G)oto to tell the Delhi caravel to go places; the client shows a goto path, but on committing it, the unit just gets a "G" icon rather than moving immediately. However, using the numeric keypad to steer, I can make use of the rest of the unit's movement points this turn. This suggests to me that it's some problem with pathfinding.

I am suspicious that is_unit_being_refueled() understands the "Coast" refuel semantics but can_unit_survive_at_tile() shows no sign of doing so. I haven't checked to see where else it might matter (fuel maps?)

#2 Updated by Marko Lindqvist over 2 years ago

  • Category set to General

I can't easily reproduce wrong behavior. Sure, sometimes Caravel waits on refuel point when it would have remaining movement points to move out to sea, but as far as I can see only when it as an fueled unit is supposed to keep itself refueled.

Jacob Nevins wrote:

I am suspicious that is_unit_being_refueled() understands the "Coast" refuel semantics but can_unit_survive_at_tile() shows no sign of doing so. I haven't checked to see where else it might matter (fuel maps?)

However, this can_unit_survive_at_tile() behavior is clearly a bug, regardless if it plays any role here -> Bug #869622

#3 Updated by Marko Lindqvist 6 months ago

  • Sprint/Milestone changed from 3.0.0 to 3.0.1

#4 Updated by Marko Lindqvist 4 months ago

  • Sprint/Milestone changed from 3.0.1 to 3.0.2

#5 Updated by Marko Lindqvist about 2 months ago

  • Sprint/Milestone changed from 3.0.2 to 3.0.3

#6 Updated by Marko Lindqvist 3 days ago

  • Sprint/Milestone changed from 3.0.3 to 3.0.4

Also available in: Atom PDF