bugFreeciv - Bugs: bug #20625, aircrafts use moving carriers when...

 
 
Show feedback again

bug #20625: aircrafts use moving carriers when calculating goto routes

Submitted by:  None
Submitted on:  Sat 16 Mar 2013 03:59:19 PM UTC  
 
Category: NoneSeverity: 3 - Normal
Priority: 5 - NormalStatus: Fixed
Assigned to: Jacob Nevins <jtn>Originator Email: -unavailable-
Open/Closed: ClosedRelease: 2.3.3
Operating System: AnyPlanned Release: 2.4.3,2.5.0,2.6.0

Add a New Comment (Rich MarkupRich Markup):
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Sat 01 Mar 2014 12:19:06 PM UTC, SVN revision 24593:

Do not treat transports with orders as potential refuel points in
pathfinding.

Reported anonymously.

See gna bug #20625.

(Browse SVN revision 24593)

Jacob Nevins <jtn>
Project AdministratorIn charge of this item.
Sat 01 Mar 2014 12:19:01 PM UTC, SVN revision 24592:

Do not treat transports with orders as potential refuel points in
pathfinding.

Reported anonymously.

See gna bug #20625.

(Browse SVN revision 24592)

Jacob Nevins <jtn>
Project AdministratorIn charge of this item.
Sat 01 Mar 2014 12:16:06 PM UTC, SVN revision 24591:

Do not treat transports with orders as potential refuel points in
pathfinding.

Reported anonymously.

See gna bug #20625.

(Browse SVN revision 24591)

Jacob Nevins <jtn>
Project AdministratorIn charge of this item.
Fri 28 Feb 2014 12:31:30 AM UTC, comment #2:

Based on an autogame where this patch made no difference to the AI's behaviour, and on comments like "the AI doesn't control units with orders" (is_boat_free()), I intend to commit this.

Jacob Nevins <jtn>
Project AdministratorIn charge of this item.
Sun 16 Feb 2014 01:07:19 AM UTC, comment #1:

The attached simple patch addresses this for human players -- transports are not considered refuel points if they have any orders.

However, will this upset the AI? -- does it use orders in such a way that they might include waiting for aircraft? Or does the AI not use the orders system?

(file #20073, file #20074)

Jacob Nevins <jtn>
Project AdministratorIn charge of this item.
Sat 16 Mar 2013 03:59:19 PM UTC, original submission:

when an aircraft is sent to a remote place, and the route takes several turns, landings in cities or airbases or carriers are calculated in to make the route possible.

but, such routes will include moving carriers as landing points as well. The carrier will be gone in the next move. Landing is impossible, you lose aircrafts because they run out of gas.

a moving carrier should not be considered as a landing possibility when calculating aircraft goto routes.

Anonymous

 

(Note: upload size limit is set to 1024 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

Attached Files
file #20073:  trunk-moving-carriers-no-refuel.patch added by jtn (820B - text/x-patch - trunk r24474 + test game from 2.4.2)
file #20074:  carrier-orders.sav.bz2 added by jtn (14kB - application/x-bzip - trunk r24474 + test game from 2.4.2)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by jtn (Updated the item)
  • -unavailable- added by None (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 9 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Sat 01 Mar 2014 12:04:47 PM UTCjtnStatusReady For Test=>Fixed
      Open/ClosedOpen=>Closed
    Fri 28 Feb 2014 12:31:30 AM UTCjtnAssigned toNone=>jtn
    Fri 28 Feb 2014 12:31:29 AM UTCjtnStatusNeed Info=>Ready For Test
    Sun 16 Feb 2014 10:21:49 AM UTCjtnStatusNone=>Need Info
    Sun 16 Feb 2014 10:21:34 AM UTCjtnOperating SystemMicrosoft Windows=>Any
      Planned Release=>2.4.3,2.5.0,2.6.0
    Sun 16 Feb 2014 01:07:19 AM UTCjtnAttached File-=>Added trunk-moving-carriers-no-refuel.patch, #20073
      Attached File-=>Added carrier-orders.sav.bz2, #20074
    Show feedback again

    Back to the top


    Powered by Savane 3.1-cleanup